Dan Kenigsberg has uploaded a new change for review. Change subject: stop supervdsmd before configuring libvirt ......................................................................
stop supervdsmd before configuring libvirt vdsmd is stopped, if existing, before upgrade and reconfiguration of libvirt. This used to stop supervdsmd, too. But now that supervdsmd is a first-class service, it should be explicitly stopped too. Otherwise, it would hold a stale libvirtconnection object, that is going to break on the subsequent usage. Change-Id: I92e7baa6fade0c0d111350c8e788bbefaaa3ae17 Bug-Url: https://bugzilla.redhat.com/990517 Signed-off-by: Dan Kenigsberg <dan...@redhat.com> --- M src/plugins/ovirt-host-deploy/vdsm/packages.py 1 file changed, 2 insertions(+), 0 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-host-deploy refs/changes/96/17596/1 diff --git a/src/plugins/ovirt-host-deploy/vdsm/packages.py b/src/plugins/ovirt-host-deploy/vdsm/packages.py index 2f09e7c..06c9f42 100644 --- a/src/plugins/ovirt-host-deploy/vdsm/packages.py +++ b/src/plugins/ovirt-host-deploy/vdsm/packages.py @@ -104,6 +104,8 @@ def _packages(self): if self.services.exists('vdsmd'): self.services.state('vdsmd', False) + if self.services.exists('supervdsmd'): + self.services.state('supervdsmd', False) self.packager.install(('qemu-kvm-tools',)) self.packager.installUpdate(('vdsm', 'vdsm-cli')) -- To view, visit http://gerrit.ovirt.org/17596 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I92e7baa6fade0c0d111350c8e788bbefaaa3ae17 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: master Gerrit-Owner: Dan Kenigsberg <dan...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
