Federico Simoncelli has uploaded a new change for review.
Change subject: svdsm: fix fuser import in supervdsmServer ......................................................................
svdsm: fix fuser import in supervdsmServer
Change-Id: I319a96e1357b5ed47ef5b6bcf3dec0ce678b9748 Signed-off-by: Federico Simoncelli fsimonce@redhat.com --- M vdsm/supervdsmServer.py 1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/04/13804/1
diff --git a/vdsm/supervdsmServer.py b/vdsm/supervdsmServer.py index e325155..28d5955 100755 --- a/vdsm/supervdsmServer.py +++ b/vdsm/supervdsmServer.py @@ -26,7 +26,7 @@ import errno import threading import re -import fuser +from storage import fuser from time import sleep import signal from multiprocessing import Pipe, Process
-- To view, visit http://gerrit.ovirt.org/13804 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange Gerrit-Change-Id: I319a96e1357b5ed47ef5b6bcf3dec0ce678b9748 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimonce@redhat.com
oVirt Jenkins CI Server has posted comments on this change.
Change subject: svdsm: fix fuser import in supervdsmServer ......................................................................
Patch Set 1:
Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1965/ (1/2)
-- To view, visit http://gerrit.ovirt.org/13804 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I319a96e1357b5ed47ef5b6bcf3dec0ce678b9748 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimonce@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.
Change subject: svdsm: fix fuser import in supervdsmServer ......................................................................
Patch Set 1:
Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1914/ (2/2)
-- To view, visit http://gerrit.ovirt.org/13804 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I319a96e1357b5ed47ef5b6bcf3dec0ce678b9748 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimonce@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.
Change subject: svdsm: fix fuser import in supervdsmServer ......................................................................
Patch Set 1:
Build Successful
http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit/1965/ : SUCCESS
http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/1914/ : SUCCESS
-- To view, visit http://gerrit.ovirt.org/13804 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I319a96e1357b5ed47ef5b6bcf3dec0ce678b9748 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimonce@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server
Federico Simoncelli has posted comments on this change.
Change subject: svdsm: fix fuser import in supervdsmServer ......................................................................
Patch Set 1: Verified
Verified: supervdsmServer is now up and running.
-- To view, visit http://gerrit.ovirt.org/13804 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I319a96e1357b5ed47ef5b6bcf3dec0ce678b9748 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimonce@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Federico Simoncelli fsimonce@redhat.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Yaniv Bronhaim ybronhei@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server
Yaniv Bronhaim has posted comments on this change.
Change subject: svdsm: fix fuser import in supervdsmServer ......................................................................
Patch Set 1: Looks good to me, but someone else must approve
-- To view, visit http://gerrit.ovirt.org/13804 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I319a96e1357b5ed47ef5b6bcf3dec0ce678b9748 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimonce@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Federico Simoncelli fsimonce@redhat.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Yaniv Bronhaim ybronhei@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server
Dan Kenigsberg has posted comments on this change.
Change subject: svdsm: fix fuser import in supervdsmServer ......................................................................
Patch Set 1: Looks good to me, approved
Oh goodness, we keep breaking your master branch, Federico. Sorry.
We should probably backport this fuser mess to ovirt-3.2, too.
-- To view, visit http://gerrit.ovirt.org/13804 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I319a96e1357b5ed47ef5b6bcf3dec0ce678b9748 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimonce@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Federico Simoncelli fsimonce@redhat.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Yaniv Bronhaim ybronhei@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server
Dan Kenigsberg has submitted this change and it was merged.
Change subject: svdsm: fix fuser import in supervdsmServer ......................................................................
svdsm: fix fuser import in supervdsmServer
Change-Id: I319a96e1357b5ed47ef5b6bcf3dec0ce678b9748 Signed-off-by: Federico Simoncelli fsimonce@redhat.com --- M vdsm/supervdsmServer.py 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: Yaniv Bronhaim: Looks good to me, but someone else must approve Federico Simoncelli: Verified Dan Kenigsberg: Looks good to me, approved
-- To view, visit http://gerrit.ovirt.org/13804 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged Gerrit-Change-Id: I319a96e1357b5ed47ef5b6bcf3dec0ce678b9748 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimonce@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Federico Simoncelli fsimonce@redhat.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Yaniv Bronhaim ybronhei@redhat.com Gerrit-Reviewer: oVirt Jenkins CI Server
vdsm-patches@lists.fedorahosted.org