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

danken at redhat.com danken at redhat.com
Tue Jan 20 15:01:47 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

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


multipath configurator: raise only if service is active

If multipathd is active, configuration must be reloaded
actively by vdsm, otherwise it will be reloaded once
multipathd starts.

Configuration will fail only in case multipathd is active
and cannot be reloaded.

Change-Id: Ie1e8eceaaa486f05b1ea4f1733df669b2a44682a
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1173487
Signed-off-by: Yeela Kaplan <ykaplan at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/36387
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M lib/vdsm/tool/configurators/multipath.py
1 file changed, 4 insertions(+), 3 deletions(-)

Approvals:
  Yeela Kaplan: Verified
  Dan Kenigsberg: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie1e8eceaaa486f05b1ea4f1733df669b2a44682a
Gerrit-PatchSet: 10
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yeela Kaplan <ykaplan at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer 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


More information about the vdsm-patches mailing list