Change in vdsm[master]: move monitored_paths to configure.ac

Xu He Jie xuhj at linux.vnet.ibm.com
Fri Jul 27 06:30:35 UTC 2012


Xu He Jie has posted comments on this change.

Change subject: move monitored_paths to configure.ac
......................................................................


Patch Set 4: (1 inline comment)

....................................................
File configure.ac
Line 85: AC_SUBST([vdsmmonitoredpaths],
ok, so vdsm-tool verb 'configure-core-dump' need a tools to modify vdsm.conf at runtime when coredump was disabled or enabled. Actually, I submitted a patch 'actually' before, but for keep this core-dump simple, I abandoned it by myself. If this  is worth maintianing a new-class to write config file. I will resubmit that patch. Thanks for the review!

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5d2c468c56f0c97eadacd1905c7e7def5a207963
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Xu He Jie <xuhj at linux.vnet.ibm.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Xu He Jie <xuhj at linux.vnet.ibm.com>


More information about the vdsm-patches mailing list