Change in vdsm[master]: Sometimes the test host is overloaded and that might cause a...

Saggi Mizrahi has uploaded a new change for review.

Change subject: Sometimes the test host is overloaded and that might cause a false positive with such a low timeout
......................................................................

Sometimes the test host is overloaded and that might cause a false positive with such a low timeout

Change-Id: Ibf80dfc8432b30967cd223460747095dc829878a
Signed-off-by: Saggi Mizrahi 
---
M tests/remoteFileHandlerTests.py
1 file changed, 1 insertion(+), 1 deletion(-)

git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/51/9751/1

diff --git a/tests/remoteFileHandlerTests.py b/tests/remoteFileHandlerTests.py
index 9508536..b114abf 100644
--- a/tests/remoteFileHandlerTests.py
+++ b/tests/remoteFileHandlerTests.py
@@ -38,7 +38,7 @@
                   And wash it down with a cup of hot steaming rules. Eh?."""
                # (C) BBC - The Mighty Boosh

-        self.assertEquals(self.pool.callCrabRPCFunction(1, "echo", data), data)
+        self.assertEquals(self.pool.callCrabRPCFunction(5, "echo", data), data)

def testTimeout(self):
         sleep = 5

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ibf80dfc8432b30967cd223460747095dc829878a
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi

[Jenkins CI RO]

oVirt Jenkins CI Server has posted comments on this change.

Change subject: Sometimes the test host is overloaded and that might cause a false positive with such a low timeout
......................................................................

Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibf80dfc8432b30967cd223460747095dc829878a
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi 
Gerrit-Reviewer: oVirt Jenkins CI Server

[Jenkins CI RO]

oVirt Jenkins CI Server has posted comments on this change.

Change subject: Sometimes the test host is overloaded and that might cause a false positive with such a low timeout
......................................................................

Patch Set 1:

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibf80dfc8432b30967cd223460747095dc829878a
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi 
Gerrit-Reviewer: oVirt Jenkins CI Server

[Jenkins CI RO]

oVirt Jenkins CI Server has posted comments on this change.

Change subject: Sometimes the test host is overloaded and that might cause a false positive with such a low timeout
......................................................................

Patch Set 1:

Build Successful

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

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

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibf80dfc8432b30967cd223460747095dc829878a
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi 
Gerrit-Reviewer: Barak Azulay 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Peter V. Saveliev 
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: oVirt Jenkins CI Server

[Dan Kenigsberg]

Dan Kenigsberg has submitted this change and it was merged.

Change subject: Sometimes the test host is overloaded and that might cause a false positive with such a low timeout
......................................................................

Sometimes the test host is overloaded and that might cause a false positive with such a low timeout

Change-Id: Ibf80dfc8432b30967cd223460747095dc829878a
Signed-off-by: Saggi Mizrahi 
---
M tests/remoteFileHandlerTests.py
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Dan Kenigsberg: Verified; Looks good to me, approved

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ibf80dfc8432b30967cd223460747095dc829878a
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi 
Gerrit-Reviewer: Barak Azulay 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Peter V. Saveliev 
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: oVirt Jenkins CI Server

[Dan Kenigsberg]

Dan Kenigsberg has posted comments on this change.

Change subject: Sometimes the test host is overloaded and that might cause a false positive with such a low timeout
......................................................................

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

5 seconds for the negative flow seems safe enough.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibf80dfc8432b30967cd223460747095dc829878a
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi 
Gerrit-Reviewer: Barak Azulay 
Gerrit-Reviewer: Dan Kenigsberg 
Gerrit-Reviewer: Peter V. Saveliev 
Gerrit-Reviewer: Yaniv Bronhaim 
Gerrit-Reviewer: oVirt Jenkins CI Server