10 Oct 2025 |
John Ericson | I think as I make more goals, we're seeing some underlying issue in the work infra | 20:31:03 |
John Ericson | Still have no idea what the hidden state is, though | 20:31:16 |
nulleric | thanks! | 20:31:24 |
John Ericson | No problem, people make this sort of mistake all the time :) | 20:31:48 |
nulleric | (i dont understand how matrix works yet, that channel you linked seems to not even be part of this "server") anyways i'll figure it out i guess ha | 20:35:02 |
John Ericson | nulleric: oh oops that was my bad | 21:05:14 |
John Ericson | I did do it wrong | 21:05:17 |
John Ericson | #dev:nixos.org this channel | 21:05:28 |
John Ericson | sorry! | 21:05:34 |
John Ericson | Sergei Zimmerman (xokdvium): your suggestion didn't quite work so I just std::ranges:find harder until it did :) | 21:14:54 |
John Ericson | if (auto it = std::ranges::find_if(
externalBuilders.get().begin(),
externalBuilders.get().end(),
[&](const auto & handler) {
return std::ranges::find(handler.systems, drv.platform) != handler.systems.end();
});
it != externalBuilders.get().end())
return &*it;
return nullptr;
| 21:15:48 |
John Ericson | :) | 21:15:54 |
Sergei Zimmerman (xokdvium) | Oh no why | 21:18:54 |
John Ericson | Sergei Zimmerman (xokdvium): added a commit to make it a set | 21:19:15 |
John Ericson | the inner thing | 21:19:21 |
lovesegfault | Sergei Zimmerman (xokdvium): RE https://github.com/NixOS/nix/pull/14209#discussion_r2422146488
I can make the change to include the s3 params in the cacheKey
| 21:56:29 |
lovesegfault | shouldn't be hard | 21:56:30 |
Sergei Zimmerman (xokdvium) | I think it should be done somewhat centralized and with a common abstraction for the stores. This is a pre-existing problem already for the s3 store thoughh | 21:57:18 |
Sergei Zimmerman (xokdvium) | * I think it should be done somewhat centralized and with a common abstraction for the stores. This is a pre-existing problem already for the s3 store though | 21:57:21 |
lovesegfault | ACK | 22:04:18 |
lovesegfault | pushed a new rev of https://github.com/NixOS/nix/pull/14209 if you could take another look :) | 22:06:30 |
11 Oct 2025 |
John Ericson | Sergei Zimmerman (xokdvium): oh no c97b050a6c212d0b748303080b5604309b7abdce is this the thing that does the closure that I guess is no good | 00:27:50 |
John Ericson | nevermind maybe not the same bug | 00:27:54 |
John Ericson | or at least that one is also a problem | 00:27:59 |
fzakaria | any good issues to try and tackle? Sergei Zimmerman (xokdvium) i guess you figured out that weird postgres one | 02:26:09 |
fzakaria | i'm not sure how you narrowed it down to path length; i missed that part of the RCA | 02:26:20 |
| perra23 joined the room. | 04:38:38 |
lovesegfault | Hot off the press: https://github.com/NixOS/nix/pull/14218 | 18:43:30 |
lovesegfault | Hot off the press: https://github.com/NixOS/nix/pull/14219 | 19:11:00 |
John Ericson | nevermind, that is just a fairly innocuous boolean field | 20:07:03 |