Change in vdsm[master]: multipath configurator: reload only if service is active

ybronhei at redhat.com ybronhei at redhat.com
Mon Dec 29 15:39:31 UTC 2014


Yaniv Bronhaim has posted comments on this change.

Change subject: multipath configurator: reload only if service is active
......................................................................


Patch Set 1: Code-Review+1

more right to say that over systemd "service multipathd reload" returns 127, but over initctl - 0 - when service is down. therefore, service.py throws exception and we don't get the runtimeError.

with this change we avoid the reload at all if service is down.

this bug suppose to exist over fedora for long time afaiu . please update the bugzilla and consider where this fix should be backported.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie1e8eceaaa486f05b1ea4f1733df669b2a44682a
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yeela Kaplan <ykaplan at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Yeela Kaplan <ykaplan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No


More information about the vdsm-patches mailing list