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 <vitor.l...@eldorado.org.br> Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com> Gerrit-Reviewer: Gustavo Frederico Temple Pedrosa <gustavo.pedr...@eldorado.org.br> Gerrit-Reviewer: Itamar Heim <ih...@redhat.com> Gerrit-Reviewer: Leonardo Bianconi <leonardo.bianc...@eldorado.org.br> Gerrit-Reviewer: Michal Skrivanek <michal.skriva...@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com> Gerrit-Reviewer: Roy Golan <rgo...@redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com> Gerrit-Reviewer: Vitor de Lima <vitor.l...@eldorado.org.br> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
