From a07f7dc8c0b98f76ea083145e991ed56f1cdb99a Mon Sep 17 00:00:00 2001 From: Matthew Oliver Date: Thu, 27 Apr 2017 01:03:29 +0000 Subject: [PATCH] Fix sporadic failure in TestAccountController unit test The proxy server on occasion has error limited a node by the time the test runs, causing the proxie's node_iter failing to iter out this error limited node. As the test uses a default FakeRing with no extra handoffs, on this occasion we only get 2 requests which is not enough for quorum, causing it to return a 503. This patch sets the error_suppression_interval to 0 when creating the proxy server. Meaning a node effectively isn't error_limited. Change-Id: I96cf4c4d63594f803cc1cd57e874d1624db8e249 Closes-Bug: #1682026 --- test/unit/proxy/test_server.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/test/unit/proxy/test_server.py b/test/unit/proxy/test_server.py index 8839d7e92b..48f7bc89b7 100644 --- a/test/unit/proxy/test_server.py +++ b/test/unit/proxy/test_server.py @@ -7961,7 +7961,8 @@ class TestContainerController(unittest.TestCase): class TestAccountController(unittest.TestCase): def setUp(self): - self.app = proxy_server.Application(None, FakeMemcache(), + conf = {'error_suppression_interval': 0} + self.app = proxy_server.Application(conf, FakeMemcache(), account_ring=FakeRing(), container_ring=FakeRing())