Change in vdsm[master]: ConfigFile.py: include default section wrapping comments.

danken at redhat.com danken at redhat.com
Mon May 26 12:42:24 UTC 2014


Dan Kenigsberg has submitted this change and it was merged.

Change subject: ConfigFile.py: include default section wrapping comments.
......................................................................


ConfigFile.py: include default section wrapping comments.

Since ConfigFile.py represents common operations on config files
it should also include the default comments vdsm inserts.
Only version has no default to allow different callares
have different configuration version.

Change-Id: I54b6b10d64d4a2d299e8fc783880c92c07dedd7b
Signed-off-by: Mooli Tayer <mtayer at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/27820
Reviewed-by: Yaniv Bronhaim <ybronhei at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M lib/vdsm/tool/configfile.py
M tests/toolTests.py
2 files changed, 39 insertions(+), 29 deletions(-)

Approvals:
  Yaniv Bronhaim: Looks good to me, but someone else must approve
  mooli tayer: Verified
  Dan Kenigsberg: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I54b6b10d64d4a2d299e8fc783880c92c07dedd7b
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: mooli tayer <mtayer at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: mooli tayer <mtayer at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list