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

Support per-breed definitions and inheritance in distro_signatures.json #11

Open
alanoe opened this issue Nov 14, 2014 · 4 comments
Open
Labels
enhancement New feature or request
Milestone

Comments

@alanoe
Copy link

alanoe commented Nov 14, 2014

Currently, several OS versions under a breed replicate the same fields (eg default_autoinstall). This change will minimize information duplication and decrease the signature file size.

@alanoe alanoe self-assigned this Nov 17, 2014
@jmaas
Copy link
Member

jmaas commented Jan 2, 2015

What would be great if it would be possible to override breed values down from the distro level. There are some outstanding bugs in buildiso because of differences in kopts in a single breed (opensuse/sles).

On the other hand i dont really like the json format at all, i'd rather see a more human friendly format like YAML being used; i once started prototyping this.. in this prototype i even got rid of the breed concept altogether. i should finish it and see if it would actually improve matters..

@alanoe
Copy link
Author

alanoe commented Jan 5, 2015

JM> What would be great if it would be possible to override breed values down from the distro level.

I do not understand exactly what you mean, Jörgen. distro level = distro OS version (eg sles12) level ? If so, my proposal will do what you suggest.

Currently all attributes in distro_signatures.json are already associated with an os_version. My proposal is to allow distro signatures file to also have attributes in the breed level and combine breed and os versions attributes during distro signatures file parsing. An attribute in os_version would override an attribute in breed, as Cobbler already does for distro-profile-system.

@SchoolGuy
Copy link
Member

The specification and import of this won't be mainted here but instead in the library created for it. I will thus transfer this issue there.

@SchoolGuy SchoolGuy transferred this issue from cobbler/cobbler Nov 19, 2020
@SchoolGuy SchoolGuy added the enhancement New feature or request label Nov 19, 2020
@SchoolGuy SchoolGuy added this to the v1.1.0 milestone Nov 22, 2020
@SchoolGuy
Copy link
Member

I believe that this can be solved if we have a special version which is called default in each breed which collects the default of each breed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Development

No branches or pull requests

3 participants