| 19 Dec 2023 |
 hexa | Redacted or Malformed Event | 22:11:22 |
| hexa | Redacted or Malformed Event | 22:11:37 |
| hexa | Redacted or Malformed Event | 22:13:02 |
| hexa | looks like there was a blip, but we still have 9 evaluators | 22:14:36 |
 cole-h | In reply to @infinisil:matrix.org
The eval queue is rising.. A side effect of evals taking 10m+ a pop 😅 | 22:45:30 |
 @infinisil:matrix.org | https://github.com/NixOS/nixpkgs/pull/269403 will be able to help, but it needs to get done first 🙃 | 22:46:43 |
| cole-h | I've got my eye on it. I also tried out nix-eval-jobs manually with outpaths.nix at one point but it took more than 24 hours (at which point I killed it) | 22:48:26 |
| cole-h | Maybe I was holding it wrong | 22:48:37 |
 Lily Foster | In reply to @cole-h:matrix.org
Maybe I was holding it wrong probably, as i did not experience that. i've experimented with it as well but the semantics were a little different in result. i intend to more thoroughly at least attempt to make a PoC with ofborg using it soon for comparison | 23:03:42 |
| 20 Dec 2023 |
 @janik0:matrix.org | Did ofborg die? It feels like the eval for most prs is stalled, but I don't really know where to look. | 19:42:33 |
| hexa | builders are idle | 19:49:10 |
| hexa | evals are not keeping up | 19:49:18 |
| hexa | you can check the dashboard in the room topic | 19:49:27 |
| 21 Dec 2023 |
|  symphorien joined the room. | 20:25:06 |
| 23 Dec 2023 |
|  raitobezarius changed their display name from raitobezarius to raitobezarius (DECT 2128). | 22:22:43 |
| 25 Dec 2023 |
|  trofi joined the room. | 14:43:18 |
| trofi | I would expect ofborg to add 2 listed maintainers to review nix_2_3 in https://github.com/NixOS/nixpkgs/pull/276696 . Do you know by chance why it did not add any reviewers? | 14:44:13 |
 raphi | i think because nix_2_3 is in aliases.nix so it doesn't get picked up in CI
nixVersions.nix_2_3 should work | 14:56:25 |
| trofi | Oh, that's tricky! Thank you! | 16:24:12 |
| Lily Foster | In reply to @raphi:tapesoftware.net
i think because nix_2_3 is in aliases.nix so it doesn't get picked up in CI
nixVersions.nix_2_3 should work yeah but commit message should just be for builds iirc. it uses changed paths/attrs from the outpath calculation iirc (though maybe it had other problems with determining the changed path there idk or maybe it is something alias heckery) | 16:59:51 |
| Lily Foster | if i remember, i'll double check the code later | 17:00:15 |
| Lily Foster | * yeah but commit message should just be for builds iirc. it uses changed paths/attrs from the outpath calculation iirc for maintainer pings (though maybe it had other problems with determining the changed path there idk or maybe it is something alias heckery) | 17:00:31 |
 Artturin | In reply to @lily:lily.flowers
yeah but commit message should just be for builds iirc. it uses changed paths/attrs from the outpath calculation iirc (though maybe it had other problems with determining the changed path there idk or maybe it is something alias heckery) The maintainers for all reverse feps aren't pinged | 17:01:04 |
| Lily Foster | In reply to @artturin:matrix.org
The maintainers for all reverse feps aren't pinged i guess that's true, i'm probably just misremembering then | 17:03:55 |
| Lily Foster | (i really thought it based it on files touched for some reason though rather than commit messages....) | 17:04:24 |
| Lily Foster | * (i really thought it based it on source files touched for some reason though rather than commit messages....) | 17:04:32 |
| Lily Foster | (idk am on mobile and also with family rn so i can't check code) | 17:04:49 |
| Artturin | Ofborg doesn't use the source names for anything IIRC | 17:06:03 |
| trofi | I changed it to nixVersions.nix_2_3 and ofborg still did not add anyone :( | 19:11:02 |
| 26 Dec 2023 |
| Lily Foster | In reply to @artturin:matrix.org
Ofborg doesn't use the source names for anything IIRC i'm looking at the ofborg code and it definitely just does maintainer pings for any changed outpaths that also had a file modified in the PR that corresponds to the derivations' meta.maintainers, src, pname, or version attr positions: https://github.com/NixOS/ofborg/blob/de415d372959b7e6fc6b2f6c95f0c21e5010348d/ofborg/src/maintainers.nix#L56-L64 | 01:31:56 |
