Change in vdsm[master]: Increase timeout in remoteFileHandlerTests:PoolHandlerTests....

Saggi Mizrahi has uploaded a new change for review.

Change subject: Increase timeout in remoteFileHandlerTests:PoolHandlerTests.testStop
......................................................................

Increase timeout in remoteFileHandlerTests:PoolHandlerTests.testStop

On congested hosts (like the oVirt Jenkins host) the test might fail
even though nothing is broken it is just taking longer then expected.

Change-Id: I9334cf00c37b352adad52a0587f9675cda5bd0dd
Signed-off-by: Saggi Mizrahi <smizrahi(a)redhat.com&gt;
---
M tests/remoteFileHandlerTests.py
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/37/10637/1

diff --git a/tests/remoteFileHandlerTests.py b/tests/remoteFileHandlerTests.py
index b114abf..544ec28 100644
--- a/tests/remoteFileHandlerTests.py
+++ b/tests/remoteFileHandlerTests.py
@@ -63,7 +63,7 @@
         procPath = os.path.join("/proc", str(p.process.pid))
 
         # Make sure handler is running
-        self.assertTrue(p.proxy.callCrabRPCFunction(1, "os.path.exists",
+        self.assertTrue(p.proxy.callCrabRPCFunction(4, "os.path.exists",
                                                     procPath))
         p.stop()
         test = lambda: self.assertFalse(os.path.exists(procPath))


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I9334cf00c37b352adad52a0587f9675cda5bd0dd
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <smizrahi(a)redhat.com&gt;

[Dan Kenigsberg]

Dan Kenigsberg has posted comments on this change.

Change subject: Increase timeout in remoteFileHandlerTests:PoolHandlerTests.testStop
......................................................................


Patch Set 1: Verified; Looks good to me, approved

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I9334cf00c37b352adad52a0587f9675cda5bd0dd
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <smizrahi(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com&gt;
Gerrit-Reviewer: oVirt Jenkins CI Server

[Dan Kenigsberg]

Dan Kenigsberg has submitted this change and it was merged.

Change subject: Increase timeout in remoteFileHandlerTests:PoolHandlerTests.testStop
......................................................................


Increase timeout in remoteFileHandlerTests:PoolHandlerTests.testStop

On congested hosts (like the oVirt Jenkins host) the test might fail
even though nothing is broken it is just taking longer then expected.

Change-Id: I9334cf00c37b352adad52a0587f9675cda5bd0dd
Signed-off-by: Saggi Mizrahi <smizrahi(a)redhat.com&gt;
---
M tests/remoteFileHandlerTests.py
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Vinzenz Feenstra: Looks good to me, but someone else must approve
  Dan Kenigsberg: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I9334cf00c37b352adad52a0587f9675cda5bd0dd
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <smizrahi(a)redhat.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com&gt;
Gerrit-Reviewer: oVirt Jenkins CI Server

[Vinzenz Feenstra]

Vinzenz Feenstra has posted comments on this change.

Change subject: Increase timeout in remoteFileHandlerTests:PoolHandlerTests.testStop
......................................................................


Patch Set 1: Looks good to me, but someone else must approve

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I9334cf00c37b352adad52a0587f9675cda5bd0dd
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <smizrahi(a)redhat.com&gt;
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com&gt;
Gerrit-Reviewer: oVirt Jenkins CI Server

[Jenkins CI RO]

oVirt Jenkins CI Server has posted comments on this change.

Change subject: Increase timeout in remoteFileHandlerTests:PoolHandlerTests.testStop
......................................................................


Patch Set 1:

Build Successful 

http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/544/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_unit_tests_manual_gerrit/579/ : SUCCESS

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I9334cf00c37b352adad52a0587f9675cda5bd0dd
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <smizrahi(a)redhat.com&gt;
Gerrit-Reviewer: oVirt Jenkins CI Server

[Jenkins CI RO]

oVirt Jenkins CI Server has posted comments on this change.

Change subject: Increase timeout in remoteFileHandlerTests:PoolHandlerTests.testStop
......................................................................


Patch Set 1:

Build Started http://jenkins.ovirt.org/job/vdsm_pep8_gerrit/544/ (1/2)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I9334cf00c37b352adad52a0587f9675cda5bd0dd
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <smizrahi(a)redhat.com&gt;
Gerrit-Reviewer: oVirt Jenkins CI Server

[Jenkins CI RO]

oVirt Jenkins CI Server has posted comments on this change.

Change subject: Increase timeout in remoteFileHandlerTests:PoolHandlerTests.testStop
......................................................................


Patch Set 1:

Build Started http://jenkins.ovirt.org/job/vdsm_unit_tests_manual_gerrit/579/ (2/2)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I9334cf00c37b352adad52a0587f9675cda5bd0dd
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <smizrahi(a)redhat.com&gt;
Gerrit-Reviewer: oVirt Jenkins CI Server