| 3 Oct 2025 |
Mic92 | Merged. | 07:25:45 |
getchoo | https://github.com/NixOS/nix/pull/14002 is now all green (consistently) | 08:00:40 |
Mic92 | Thanks a lot. I also opened https://github.com/NixOS/org/issues/167 | 08:16:19 |
Mic92 | so you can trigger ci runs etc. | 08:16:55 |
Mic92 | @getchoo:matrix.org: i added you now to the installer team. There is a python script repo that uploads new releases from a triggered GitHub action that accepts a hydra eval. When i tested it once it failed but it seemed to have worked at least one time. It could be that we know have to set a user agent to get past the ai scraper shield that we added in front of hydra.nixos.org? I didn't had a closer look. | 14:09:15 |
lovesegfault | thoughts on temporarily extending CI to also test withAWS = false; withCurlS3 = false and withAWS = false; withCurlS3 = true;? | 15:38:14 |
lovesegfault | I just want to reduce the chance of messing something up while we get all these parts merged | 15:38:28 |
lovesegfault | https://github.com/NixOS/nix/pull/14147 | 16:59:23 |
lovesegfault | we can revert it once we're done messing with that codepath | 16:59:46 |
| asoderlind joined the room. | 17:08:14 |
lovesegfault | cc. Sergei Zimmerman (xokdvium) on the above, I think? | 17:18:34 |
lovesegfault | the next PR in the series starts to introduce things side-by-side, so it'll be nice to have that in before I make it | 17:19:00 |
| moleksiak joined the room. | 23:07:21 |
| 4 Oct 2025 |
| aleksana 🏳️⚧️ (force me to bed after 18:00 UTC) changed their profile picture. | 08:41:10 |
| aleksana 🏳️⚧️ (force me to bed after 18:00 UTC) changed their profile picture. | 08:41:31 |
Sergei Zimmerman (xokdvium) | Mic92: | 21:33:25 |
Sergei Zimmerman (xokdvium) | * Mic92, John Ericson my suspect for https://hydra.nixos.org/build/308585653/nixlog/3 (https://github.com/NixOS/nix/issues/14141) is https://github.com/NixOS/nix/commit/e06968ec2586a9ccd18e58d1796de6d9ac628bc6 | 21:34:17 |
Sergei Zimmerman (xokdvium) | Trying to repro with sanitizers, but not much luck yet. Seems to be fucky UB. | 21:35:54 |
Sergei Zimmerman (xokdvium) | I think I've run into a very fucky store path corruption as well while dogfooding master. Must be the same root cause | 21:47:58 |
Sergei Zimmerman (xokdvium) | Or was it https://github.com/NixOS/nix/pull/14022/commits maybe? | 22:12:01 |
Sergei Zimmerman (xokdvium) | Alternate theory. There's UB somewhere in the code base and with the switch to -fno-semantic-interposition GCC is happily optimising the heck out of things. | 23:42:23 |
| 5 Oct 2025 |
Mic92 | @xokdvium:matrix.org: would that be catched if someone runs nix with ubsan in their hosy daemon? | 04:44:34 |
Mic92 | Sergei Zimmerman (xokdvium): https://github.com/NixOS/nix/pull/14155 | 05:31:15 |
Mic92 | for sunsetting mergify | 05:31:23 |
| Miku joined the room. | 09:31:43 |
Sergei Zimmerman (xokdvium) | Ok managed to hit this locally at least once. My suspicion is that it's a use-after-free and with musl those blow up more predictably. | 12:29:17 |
Taeer Bar-Yam | don't we run things through ASAN at some point? | 14:04:58 |
Sergei Zimmerman (xokdvium) | Yeah we do now, but that's not enough it seems | 14:05:22 |
Sergei Zimmerman (xokdvium) | I think clang-tidy managed to dig up something relevant:
../src/libstore/build/derivation-check.cc:85:35: warning: Dereference of undefined pointer value [clang-analyzer-core.NullDereference]
85 | if (checks.maxSize && info.narSize > *checks.maxSize)
../src/libstore/build/derivation-check.cc:117:31: warning: 4th function call argument is an uninitialized value [clang-analyzer-core.CallAndMessage]
117 | throw BuildError(
../src/libstore/build/derivation-check.cc:181:45: warning: 2nd function call argument is an uninitialized value [clang-analyzer-core.CallAndMessage]
181 | if (auto outputChecks = get(checksPerOutput, outputName))
| 14:08:22 |
Sergei Zimmerman (xokdvium) | Yeah turned out to be false-positives | 19:22:42 |