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

Use elemental's build-iso command to build the ISO #96

Merged
merged 1 commit into from
Apr 26, 2022

Conversation

davidcassany
Copy link
Contributor

@davidcassany davidcassany commented Apr 25, 2022

Signed-off-by: David Cassany dcassany@suse.com

Signed-off-by: David Cassany <dcassany@suse.com>
@davidcassany davidcassany force-pushed the move_to_build-iso_command branch from c37dfb2 to 637d9aa Compare April 25, 2022 07:58
@mudler
Copy link
Contributor

mudler commented Apr 26, 2022

Do we still need to build a config file?

@davidcassany
Copy link
Contributor Author

Do we still need to build a config file?

You mean the manifest embedded inside the Dockerfile? if so yes, there are no defaults set for the included packages as there is no Luet default configuration too. Note that those packages relay on a Luet repository, so I am not sure having a default for everything is that good (I mean packages list and luet repositories). Anyway this is something simple to add if we want, it is all configurable via manifest and flags, so adding some extra defaults is trivial.

@mudler
Copy link
Contributor

mudler commented Apr 26, 2022

Do we still need to build a config file?

You mean the manifest embedded inside the Dockerfile? if so yes, there are no defaults set for the included packages as there is no Luet default configuration too. Note that those packages relay on a Luet repository, so I am not sure having a default for everything is that good (I mean packages list and luet repositories). Anyway this is something simple to add if we want, it is all configurable via manifest and flags, so adding some extra defaults is trivial.

I thought was already like this so we could get rid of the base file as there are no big customization here (just the iso overlay, which I thought you could pass with some arg), at the end a full manifest could be still passed by in case of overrides. Nevertheless looks good anyway, it's a nice cutdown!

@davidcassany
Copy link
Contributor Author

davidcassany commented Apr 26, 2022

I thought was already like this so we could get rid of the base file as there are no big customization here (just the iso overlay, which I thought you could pass with some arg), at the end a full manifest could be still passed by in case of overrides. Nevertheless looks good anyway, it's a nice cutdown!

I did not add these defaults for everything as I had in mind the idea of having it always executed on top of a generic manifest and apply all little customizations on top of it form the command line. I also thought about having a default --config-dir value set to . or something like so we load the manifest from the current folder (even this does not really apply on this os2 setup, but nice from a cOS-Toolkit PoV). Anyway the main reason to not add these defaults is having to hardcode the luet repository URL. To me it feels odd hardcoding it inside elemental, but we can easily do a follow up PR for that and discuss it, I do not have any technical reason against it.

@davidcassany davidcassany merged commit 4b350e0 into master Apr 26, 2022
@davidcassany davidcassany deleted the move_to_build-iso_command branch April 26, 2022 09:33
mudler pushed a commit that referenced this pull request Jun 3, 2022
Co-authored-by: Itxaka <igarcia@suse.com>
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