From Dan Kenigsberg danken@redhat.com:
Dan Kenigsberg has submitted this change and it was merged.
Change subject: confmeta: Add confmeta module ......................................................................
confmeta: Add confmeta module
This module handles configuration files managed by vdsm, using a metadata comments header.
Example file:
# This file is managed by vdsm # description... #REVISION: 3 #PRIVATE: NO
file contents...
Example usage:
>>> from vdsm.tool import confmeta >>>> confmeta.read_metadata(path) ConfigMetadata(revision=3, private=False)
This module will be used to manage lvmlocal.conf file, and later will replace the system used to manage multiapth.conf.
Change-Id: Icbd590853ca0acc47acd7823daf39931cd617638 Bug-Url: https://bugzilla.redhat.com/1374545 Signed-off-by: Nir Soffer nsoffer@redhat.com --- M lib/vdsm/tool/Makefile.am A lib/vdsm/tool/confmeta.py M tests/Makefile.am A tests/tool_confmeta_test.py M vdsm.spec.in 5 files changed, 259 insertions(+), 1 deletion(-)
Approvals: Piotr Kliczewski: Looks good to me, but someone else must approve Nir Soffer: Verified; Passed CI tests Dan Kenigsberg: Looks good to me, approved
Objections: Yaniv Bronhaim: I would prefer that you didn't submit this Jenkins CI: Failed CI tests
vdsm-patches@lists.fedorahosted.org