Change in vdsm[master]: Removing libvirt_configure from pre-start and ask for manual...

ybronhei at redhat.com ybronhei at redhat.com
Thu Oct 3 16:59:53 UTC 2013


Yaniv Bronhaim has posted comments on this change.

Change subject: Removing libvirt_configure from pre-start and ask for manual run
......................................................................


Patch Set 2:

(2 comments)

Thanks!

....................................................
File init/vdsmd_init_common.sh.in
Line 205: case "$1" in
Line 206:     --pre-start)
Line 207:         run_tasks " \
Line 208:             run_init_hooks gencerts check_libvirt_configure \
Line 209:             reconfigure_sanlock syslog_available nwfilter dummybr \
host-deploy uses vdsm-tool to reconfigure that does only the libvirt part. I intend to do it separately to this patch
Line 210:             load_needed_modules tune_system mkdirs test_space test_lo \
Line 211:             test_conflicting_conf"
Line 212:         ;;
Line 213:     --post-stop)


....................................................
File lib/vdsm/tool/libvirt_configure.sh.in
Line 196:     # - not configured
Line 197:     #
Line 198:     if [ "${force_reconfigure}" != "--force" ] && \
Line 199:       ! [ -f "${FORCE_RECONFIGURE}" ] && \
Line 200:       is_already_configured "${lconf}" "${qconf}" "${ldconf}" "${glconf}"; then
could be a mess.. thanks for catching
Line 201:         echo "Not forcing reconfigure"
Line 202:         return 0
Line 203:     fi
Line 204: 


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Icf70a749454ea341d5b52220e16e9567b90431a0
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybronhei 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: Zhou Zheng Sheng <zhshzhou at linux.vnet.ibm.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list