| 4 Dec 2022 |
Rick (Mindavi) | For stateful things like DBs it is hard to do things right | 23:22:45 |
raitobezarius | In reply to @ronnypfannschmidt:matrix.org Safe rollback needs control over state transition, including db migration That's a necessary condition, not a sufficient one | 23:23:02 |
Rick (Mindavi) | And upstream has to support control over db migrations if you want to do anything about it | 23:23:20 |
Rick (Mindavi) | Or you need to pin a specific version of the app to a specific stateVersion | 23:23:52 |
Ronny | Well, the default nextcloud module does auto migration snd fake password uogrades, so it's problematic anyway | 23:24:15 |
Rick (Mindavi) | Which is how postgresql supports stateVersion | 23:24:24 |
Rick (Mindavi) | But they don't break the db on patch releases... | 23:24:52 |
Ronny | As for specific versions, i pinned, didn't protect | 23:24:58 |
Rick (Mindavi) | Hmm, I thought it was a version upgrade that broke it | 23:25:36 |
Rick (Mindavi) | Sorry, what happened then again? 😅 | 23:25:49 |
raitobezarius | In reply to @ronnypfannschmidt:matrix.org As for specific versions, i pinned, didn't protect I don't think we expose patch-level versions of Nextcloud | 23:25:51 |
Ronny | Major version was pinned, patch had a migration | 23:26:18 |
raitobezarius | In reply to @rick:matrix.ciphernetics.nl Sorry, what happened then again? 😅 Deployed a new patch-level version, rollbacked and I guess NC got confused because it was trying to do DB queries that don't exist or are not compatible with the patch level - 1 version | 23:26:22 |
raitobezarius | Classical stuff that upstream pulls out because rollback paths are not tested by them on that regard | 23:26:57 |
raitobezarius | (and there must be some data to exercise this stuff) | 23:27:06 |
Ronny | Granular state see necessary to manage bad apps | 23:27:11 |
raitobezarius | But the good news is even with control over your DB schema, it won't change anything with Nextcloud ;) | 23:27:17 |
raitobezarius | There's many migration bugs lurking around with plugins and WebAuthn, etc. | 23:27:28 |
Rick (Mindavi) | I think it's best to always run a backup before a system update | 23:27:43 |
Rick (Mindavi) | Of stateful data | 23:27:51 |
raitobezarius | A good step IMHO would be https://github.com/NixOS/nixops/pull/1245 | 23:28:15 |
| 5 Dec 2022 |
Robert Hensing (roberth) | reminder: I can't make the meeting today | 10:20:16 |
infinisil | @room: The next meeting will take place soon, we will continue discussing issues in https://github.com/nixpkgs-architecture/simple-package-paths - meeting link - live stream - meeting notes | 15:21:47 |
infinisil | John Ericson tomberek ^ | 15:34:05 |
infinisil | Meeting notes: https://discourse.nixos.org/t/2022-12-05-nixpkgs-architecture-team-meeting-20/23740 | 16:09:51 |
nbp | Stupid question, has anybody contacted github asking what they think of the problem / solution? After all being one of the most contributed repository, this might be impactful for them. | 16:14:29 |
infinisil | nbp: Not a stupid question, that's a good idea! | 16:43:01 |
infinisil | Related to the recent collaboration: https://discourse.nixos.org/t/nixos-github-collaboration/23432 | 16:43:43 |
| 7 Dec 2022 |
infinisil | If somebody wants to help out, I'm in https://meet.jit.si/nixpkgs-architecture, working on the rust tool for Nix file renames in https://github.com/nixpkgs-architecture/simple-package-paths/pull/22 | 19:07:08 |
| 8 Dec 2022 |
Wanja Hentze | Hey, I know Rust, I think I can help | 09:53:38 |