-
Notifications
You must be signed in to change notification settings - Fork 107
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
convert bootstrap iso to a nixos-generator format #455
base: main
Are you sure you want to change the base?
Conversation
c2b992e
to
c81856f
Compare
Possibly relevant: took a crack in my |
c81856f
to
30ffa0b
Compare
Why not? IIRC, this was originally exposed to
I'm hoping to test it soon -- I have a remote server to re-provision coming up when I get some downtime. |
After testing this change in our fork, trying to fix iso builds, we noticed that all hosts seem to build iso configurations after this commit: I don't understand yet how the changes in this PR could affect all hosts and not just
|
@teutat3s I ran into a similar moment of confusion in my own configs -- however, I have not been tracking this branch. The fix, in my case, was removing the ISO module from Based on this line in your source repo, it looks like you also have it included in |
@montchr thank you for the feedback. Yeah the module was included before, we sticked with DevOS defaults of |
After testing this for a while in our CI, I'd like to confirm this works fine. What else is needed to move this forward? We needed the following two changes in our https://git.b12f.io/pub-solar/os/commit/b7ba1b7ba9b87c7a60e3f2fe4772d8579b0ed711
https://git.b12f.io/pub-solar/os/commit/0fbabc23b921fa8ac8ac5647d1ff6b5a360b70c4
|
fixes #450
I'm not sure if this should then be exported under
nixosModules
, and also I'm not sure how this will integrate with nix-community/nixos-generators#148.And this is currently untested.