Change in ovirt-log-collector[master]: conf: use ovirt-engine-lib configfile

sbonazzo at redhat.com sbonazzo at redhat.com
Thu Oct 24 15:02:23 UTC 2013


Sandro Bonazzola has submitted this change and it was merged.

Change subject: conf: use ovirt-engine-lib configfile
......................................................................


conf: use ovirt-engine-lib configfile

Use ovirt-engine-lib configfile module for reading
configuration files.
The previous implementation had an issue if a variable
was defined recursively in more than one config file.

Change-Id: I1f47875a279953bae783407dca41102904a70061
Bug-Url: http://bugzilla.redhat.com/1022557
Signed-off-by: Sandro Bonazzola <sbonazzo at redhat.com>
---
M ovirt-log-collector.spec.in
M src/Makefile.am
M src/__main__.py
D src/util.py
4 files changed, 6 insertions(+), 110 deletions(-)

Approvals:
  Sandro Bonazzola: Verified; Looks good to me, approved
  Kiril Nesenko: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I1f47875a279953bae783407dca41102904a70061
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-log-collector
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Alex Lourie <alourie at redhat.com>
Gerrit-Reviewer: Keith Robertson <kroberts at redhat.com>
Gerrit-Reviewer: Kiril Nesenko <knesenko at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list