From Dan Kenigsberg danken@redhat.com:
Dan Kenigsberg has posted comments on this change.
Change subject: confmeta: Add confmeta module ......................................................................
Patch Set 10: Code-Review+1
(1 comment)
https://gerrit.ovirt.org/#/c/70583/10//COMMIT_MSG Commit Message:
Line 21: Example usage: Line 22: Line 23: >>> from vdsm.tool import confmeta Line 24: >>>> confmeta.read_metadata(path) Line 25: ConfigMetadata(revision=3, private=False)
This module handle lvm.conf, and should replace the code in multipath.conf.
Yaniv and Mooli, I tend to agree with Nir: his new configurator intends only to mark configuration files as "ours" or "private". It is quite distinct from editing ini files. maybe the name "configfile" is too broad. For example, multipath.conf is also a configuration file, and it is justly not handled by the configfile configurator. Line 26: Line 27: This module will be used to manage lvmlocal.conf file, and later will Line 28: replace the system used to manage multiapth.conf. Line 29:
vdsm-patches@lists.fedorahosted.org