Change in ovirt-engine[master]: packaging: Add support for ${...} in local config

juan.hernandez at redhat.com juan.hernandez at redhat.com
Tue Mar 5 15:34:56 UTC 2013


Juan Hernandez has submitted this change and it was merged.

Change subject: packaging: Add support for ${...} in local config
......................................................................


packaging: Add support for ${...} in local config

This patch adds support for references to parameters in the local
configuration file. This is convenient in order to simplify the
packaging of other applications, like reports, as with this support they
don't have to modify the existing configuration files, but add new ones
in the /etc/ovirt-engine/engine.conf.d directory. For example, in order
to add the reports application there is no need to modify the existing
engine.conf file to add the following:

  ENGINE_APPS=engine.ear reports.war

Instead of that a file can be created in engine.conf.d with the
following content:

  ENGINE_APPS=${ENGINE_APPS} reports.war

Change-Id: Ib8ce501241ed1a937db24e1371e2d4227679ae74
Signed-off-by: Juan Hernandez <juan.hernandez at redhat.com>
---
M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/LocalConfig.java
M packaging/fedora/engine-service.py.in
M packaging/fedora/spec/ovirt-engine.spec.in
3 files changed, 141 insertions(+), 27 deletions(-)

Approvals:
  Juan Hernandez: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/12718
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib8ce501241ed1a937db24e1371e2d4227679ae74
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>



More information about the Engine-commits mailing list