| 13 Nov 2023 |
 raitobezarius | this stuff doesn't possess any test in nixpkgs | 14:33:55 |
| raitobezarius | otherwise everything else should be covered by tests (which uncovered already issues in the PR) | 14:34:12 |
| raitobezarius | I want to cry, people made stupid defaults | 14:43:21 |
| raitobezarius | Mobilizon has a default database of mobilizon_prod, I switch it back to user as a default, this will make all existing deployments explode | 14:43:40 |
| raitobezarius | * Mobilizon has a default database of mobilizon_prod, if I switch it back to user as a default, this will make all existing deployments explode | 14:45:09 |
 bendlas | oof ... so add ensureDatabaseOwners = {}; instead, or switch to mobilizon database name, pending stateVersion == 23.11? | 14:49:59 |
| raitobezarius | I'm trying to evaluate the blast radius | 14:52:35 |
| raitobezarius | And we can think of potential solutions | 14:52:40 |
| raitobezarius | So far, Mobilizon is like this, SourceHut possessā¦ multiple databases | 14:52:54 |
| raitobezarius | (metasrht, buildsrht, gitsrht, etc, etc.) | 14:53:00 |
| raitobezarius | But I think they are operated under the same user | 14:53:06 |
| raitobezarius | OK, that's it in terms of evaluation errors | 14:53:27 |
| raitobezarius | Mobilizon and SourceHut | 14:53:32 |
| raitobezarius | 
Download image.png | 14:53:43 |
| raitobezarius | gro BRRR EPYC | 14:53:44 |
| bendlas | I mean, the cleanest solution would be services.postgresql.ensureDatabases.*.owner, but unfortunately that has also been taken from us, no thanks to ensureDatabases being a list of strings š„² | 14:54:26 |
| raitobezarius | Yep, I think I agree with that | 14:54:41 |
| raitobezarius | This is for 24.05 | 14:54:48 |
| raitobezarius | My only objective here is to minimize the amount of breakages we will have to fix during ZHF | 14:55:03 |
| raitobezarius | It's likely we will have remnants issues during the beta of 23.11 | 14:55:58 |
| raitobezarius | If we can have them down to 1 or 2 modules, that's enough for me | 14:56:06 |
| raitobezarius | # pgbouncer is in crash loop.
# dex-oidc fails.
# coder doesn't build.
# invidious is in crash loop
# odoo is in crash loop.
so far
| 15:29:22 |
| raitobezarius | in addition to the 2 evaluation errors | 15:29:29 |
| raitobezarius | the rest is passing | 15:29:54 |
| raitobezarius | so blast radius is 2 eval errors, 3 crash loops, 1 non-build, 1 proper failure | 15:30:07 |
| raitobezarius | let's compare to master | 15:30:18 |
| bendlas | prepared a section, but still have to test & verify https://github.com/NixOS/nixpkgs/pull/267238 | 15:37:40 |
| bendlas | ugh, can I not do an unlinked heading? heading in line 52 does not have an id | 15:39:33 |
| raitobezarius | yep everything must have a link | 15:43:50 |
| raitobezarius | I'd say this is pretty good already | 15:43:54 |
