Skip to content

https://github.com/NixOS/hydra/pull/1517 + test fixes#1579

Merged
Ericson2314 merged 12 commits intoNixOS:masterfrom
obsidiansystems:queue-runner-v2
Mar 17, 2026
Merged

https://github.com/NixOS/hydra/pull/1517 + test fixes#1579
Ericson2314 merged 12 commits intoNixOS:masterfrom
obsidiansystems:queue-runner-v2

Conversation

@Ericson2314
Copy link
Member

This is not my work! This is 99% #1517 by @helsinki-systems. There is just a little bit of test fixes and merge conflict systems from my coworkers and I on top. All credit to @Conni2461 and the rest of his team!

Conni2461 and others added 10 commits March 11, 2026 18:43
The new Rust queue-runner doesn't create gcroots on the central runner —
the builder copies outputs to a remote store and cleans up its own roots
at shutdown. Comment out the test assertion that expected a gcroot to exist.

Co-authored-by: Artemis Tosini <artemis.tosini@obsidian.systems>
The Rust queue-runner uses a TOML config file and different environment
variables than the C++ version, so we generate a config.toml from the foreman
environment and pass HYDRA_DATABASE_URL for the database connection.
Use `Config::remote_store_addr` for dest store.
@Ericson2314 Ericson2314 enabled auto-merge March 17, 2026 15:33
@Ericson2314 Ericson2314 added this pull request to the merge queue Mar 17, 2026
@Ericson2314 Ericson2314 removed this pull request from the merge queue due to a manual request Mar 17, 2026
This needed a change to not just build part of Hydra after the build was
split up.
It makes the build even on my machine really slow. On GitHub Actions, I
shudder to think how long it would take.

Once PRs are Hydra are no longer CI'd with GitHub Actions, we can
reconsider this ;).
@Ericson2314 Ericson2314 enabled auto-merge March 17, 2026 16:02
@Ericson2314 Ericson2314 added this pull request to the merge queue Mar 17, 2026
Merged via the queue into NixOS:master with commit d92a6b6 Mar 17, 2026
2 checks passed
@Ericson2314 Ericson2314 deleted the queue-runner-v2 branch March 17, 2026 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants