Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Flakify the system #189

Merged
merged 2 commits into from
Apr 18, 2024
Merged

Flakify the system #189

merged 2 commits into from
Apr 18, 2024

Conversation

lheckemann
Copy link
Member

This should hopefully make it easy to move the builds of this machine's config to hydra.nixos.org and its boot process to the nix-netboot-serve instance on netboot.nixos.org.

Related: NixOS/infra#416

This should hopefully make it easy to move the builds of this
machine's config to hydra.nixos.org and its boot process to the
nix-netboot-serve instance on netboot.nixos.org.
@delroth delroth merged commit c075aa9 into NixOS:master Apr 18, 2024
@lheckemann lheckemann deleted the flake branch April 18, 2024 18:18
@cole-h
Copy link
Member

cole-h commented Apr 18, 2024

It would be great if we could keep the current deploy setup working in this transition period...

https://buildkite.com/grahamc/nix-community-aarch64-build-box/builds/835

@delroth
Copy link
Contributor

delroth commented Apr 18, 2024

The Hydra jobset is already up and building, does there need to be a transition period? https://hydra.nixos.org/eval/1805747

@cole-h
Copy link
Member

cole-h commented Apr 18, 2024

I guess my stance is: it'd be nice to be able to keep the "old" way working in case the "new" way doesn't, because I'd like to keep ofborg's aarch64-linux CI working.

If you aren't worried about this new setup causing issues then 🤷 guess it's unnecessary.

@@ -210,7 +177,13 @@ in makeNetboot {
environment.etc.host-nix-channel.source = pkgs.path;
})

({pkgs, ...}: {
{
options.ofborg.package = lib.mkOption {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

a little bit strange that this lib is from outer scope, and why an inline module in imports here instead of putting it directly in the module below?

it does match the surrounding code style, just feels really weird.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah all these little modules should probably either be smooshed together or moved into their own files.

@@ -0,0 +1,14 @@
{
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

heck yeah 14 line flake. you love to see it honestly.

@lheckemann
Copy link
Member Author

@cole-h we haven't destroyed the old machine, we can revert the relevant commits and set the netboot URL back if this goes wrong.

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.

4 participants