feat: add flake-parts installer image options - #1223
Conversation
42c513a to
be79162
Compare
|
Marking this as ready for review, since the install script is implemented & I'm happy with the way it's configured |
be79162 to
e065bac
Compare
|
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/offline-automated-installation-images-for-disko/75953/1 |
|
How about some tests? |
|
Tests is on the todo above, I don't have much experience with CI, and wanted to gauge if this is something desirable before continuing with docs & tests |
|
I had actually started working on implementing a similar feature a while ago (if you want, you can check out an old version of what I had been working on here). I had gotten stuck trying to write tests for the functionality that I had added. Specifically, I thought that I needed this Nixpkgs pull request that adds a More recently, I figured out workarounds that would allow me to write correct tests even though the
Since you have been working on a similar feature already, I would love it if you could take a look at my pull request and share your thoughts. Also, sorry for coming out of nowhere and submitting a conflicting PR. I wish that I had submitted my work as a WIP pull request without tests a while ago so that we both didn’t end up putting in a bunch of duplicated effort. |
|
You're fine! Unfortunately life has gotten pretty busy for me, and I don't have much mental bandwidth to spare. So I won't be working on this for the foreseeable future |
Creating an offline installer configuration is already demonstrated in the disko docs, however it's pretty dense and I think a module would be a great replacement for copying that large chunk of code from the docs
This PR adds the
disko-installoption set to the flake-parts module, which generates installer images with dependencies & install script includedMinimal config:
Exhaustive config:
TODO
This works to generate installer images (I have tested with
iso-installer) with ainstall-nixos-unattendedscript, however it still needs work:listOf str