Omer Frenkel has submitted this change and it was merged. Change subject: core: Manage OS properties in a repository ...................................................................... core: Manage OS properties in a repository This is the implementation of the OS info according to http://www.ovirt.org/OS_info This feature enables ovirt to configure all its OS info in one place, out of the code, and in a way admins could extend it or easily configure it. It aims to merge all configuration items from the Config (currently vdc_options table) and the VmOsType.java enum to one file-based repository. This patch contains: * The default repo config file osinfo-defaults.properties * A loading logic to support multiple properties file loading * A Repository which holds an inner tree with all the configuration * A backend query to expose the data to UI Change-Id: I50384bf3b6763fa7a34badc2449e06242186a858 Signed-off-by: Roy Golan <rgolan@redhat.com> --- M Makefile M backend/manager/dbscripts/upgrade/pre_upgrade/0000_config.sql M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/Backend.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/OsRepositoryQuery.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/osinfo/MapBackedPreferences.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/osinfo/OsRepository.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/osinfo/OsRepositoryImpl.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/OsQueryParameters.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/VdcQueryType.java A backend/manager/modules/common/src/test/java/org/ovirt/engine/core/common/osinfo/OsRepositoryImplTest.java A backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/OsLoader.java A backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/osinfo/OsInfoPreferencesLoader.java A backend/manager/modules/utils/src/test/java/org/ovirt/engine/core/utils/osinfo/OsInfoPreferencesLoaderTest.java A backend/manager/modules/utils/src/test/resources/osinfo.conf.d/00-osinfo-test.properties A backend/manager/modules/utils/src/test/resources/osinfo.conf.d/10-osinfo-test.properties M ovirt-engine.spec.in A packaging/conf/osinfo-defaults.properties 18 files changed, 1,188 insertions(+), 0 deletions(-) Approvals: Omer Frenkel: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/15080 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I50384bf3b6763fa7a34badc2449e06242186a858 Gerrit-PatchSet: 18 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Roy Golan <rgolan@redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl@redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas@redhat.com> Gerrit-Reviewer: Daniel Erez <derez@redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel@redhat.com> Gerrit-Reviewer: Roy Golan <rgolan@redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server