Alon Bar-Lev has posted comments on this change. Change subject: core: New OS for IBM POWER support ......................................................................
Patch Set 15: We put single symlink of defaults to make it simple for our mechanism to load the entire configuration. The assumption is that there is one produce hence one default so we can add this abnormality at /etc/ without managing additions and removals of configuration files. If the list of product defaults is going to be dynamic we need to read two locations, first /usr/share/ovirt-engine/os-info/*.properties and then /etc/ovirt-engine/osinfo.conf.d/*.properties, this way you can add/remove defaults as you like without effecting packaging. Until we do so, please merge these two files and proceed with this change, in future you will be able to split based on above logic. -- To view, visit http://gerrit.ovirt.org/18220 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I35298c31cfa55f212c2d8b4016ee3019d7b28c40 Gerrit-PatchSet: 15 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vitor de Lima <[email protected]> Gerrit-Reviewer: Alon Bar-Lev <[email protected]> Gerrit-Reviewer: Gustavo Frederico Temple Pedrosa <[email protected]> Gerrit-Reviewer: Itamar Heim <[email protected]> Gerrit-Reviewer: Leonardo Bianconi <[email protected]> Gerrit-Reviewer: Michal Skrivanek <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: Roy Golan <[email protected]> Gerrit-Reviewer: Sandro Bonazzola <[email protected]> Gerrit-Reviewer: Vitor de Lima <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
