Change in ovirt-engine[master]: core: Manage OS properties in a repository

ofrenkel at redhat.com ofrenkel at redhat.com
Tue Jun 18 10:18:32 UTC 2013


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 at 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 at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list