Saggi Mizrahi has uploaded a new change for review.
Change subject: Read multipath.conf with supervdsm instead of SUDO ......................................................................
Read multipath.conf with supervdsm instead of SUDO
Change-Id: Iabe33d5a4493c7506ce86a203bbd75dde0fa0922 Signed-off-by: Saggi Mizrahi smizrahi@redhat.com --- M tests/miscTests.py M vdsm/storage/misc.py M vdsm/storage/multipath.py M vdsm/supervdsmServer.py 4 files changed, 8 insertions(+), 57 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/95/3595/1 -- To view, visit http://gerrit.ovirt.org/3595 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange Gerrit-Change-Id: Iabe33d5a4493c7506ce86a203bbd75dde0fa0922 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Saggi Mizrahi smizrahi@redhat.com
Dan Kenigsberg has posted comments on this change.
Change subject: Read multipath.conf with supervdsm instead of SUDO ......................................................................
Patch Set 1: Looks good to me, but someone else must approve
-- To view, visit http://gerrit.ovirt.org/3595 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: Iabe33d5a4493c7506ce86a203bbd75dde0fa0922 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com
Igor Lvovsky has posted comments on this change.
Change subject: Read multipath.conf with supervdsm instead of SUDO ......................................................................
Patch Set 1: Looks good to me, but someone else must approve
-- To view, visit http://gerrit.ovirt.org/3595 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: Iabe33d5a4493c7506ce86a203bbd75dde0fa0922 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Igor Lvovsky ilvovsky@redhat.com
Wenchao Xia has posted comments on this change.
Change subject: Read multipath.conf with supervdsm instead of SUDO ......................................................................
Patch Set 1: Looks good to me, but someone else must approve
-- To view, visit http://gerrit.ovirt.org/3595 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: Iabe33d5a4493c7506ce86a203bbd75dde0fa0922 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Igor Lvovsky ilvovsky@redhat.com Gerrit-Reviewer: Wenchao Xia xiawenc@linux.vnet.ibm.com
Dan Kenigsberg has posted comments on this change.
Change subject: Read multipath.conf with supervdsm instead of SUDO ......................................................................
Patch Set 1: Verified; Looks good to me, approved
-- To view, visit http://gerrit.ovirt.org/3595 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: Iabe33d5a4493c7506ce86a203bbd75dde0fa0922 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Igor Lvovsky ilvovsky@redhat.com Gerrit-Reviewer: Wenchao Xia xiawenc@linux.vnet.ibm.com
Dan Kenigsberg has submitted this change and it was merged.
Change subject: Read multipath.conf with supervdsm instead of SUDO ......................................................................
Read multipath.conf with supervdsm instead of SUDO
Change-Id: Iabe33d5a4493c7506ce86a203bbd75dde0fa0922 Signed-off-by: Saggi Mizrahi smizrahi@redhat.com --- M tests/miscTests.py M vdsm/storage/misc.py M vdsm/storage/multipath.py M vdsm/supervdsmServer.py 4 files changed, 8 insertions(+), 57 deletions(-)
Approvals: Wenchao Xia: Looks good to me, but someone else must approve Dan Kenigsberg: Verified; Looks good to me, approved Igor Lvovsky: Looks good to me, but someone else must approve
-- To view, visit http://gerrit.ovirt.org/3595 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged Gerrit-Change-Id: Iabe33d5a4493c7506ce86a203bbd75dde0fa0922 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Igor Lvovsky ilvovsky@redhat.com Gerrit-Reviewer: Wenchao Xia xiawenc@linux.vnet.ibm.com
vdsm-patches@lists.fedorahosted.org