From 5ccee2ba51b060c6b8eb4225d1eb7d3e46e693f5 Mon Sep 17 00:00:00 2001 From: Vasant Patil Date: Mon, 27 Jul 2020 14:29:17 -0700 Subject: [PATCH] Updated comment to reflect the code and the reason behind it RB=2204748 G=lnos-reviewers R=pchaudhary,pmao,samaity,zxu A=zxu --- tests/conftest.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/tests/conftest.py b/tests/conftest.py index 6b44b1d713..607acbcb07 100644 --- a/tests/conftest.py +++ b/tests/conftest.py @@ -172,7 +172,10 @@ def __init__(self, name=None, imgname=None, keeptb=False, fakeplatform=None): self.natd = ['natsyncd', 'natmgrd'] self.alld = self.basicd + self.swssd + self.syncd + self.rtd + self.teamd + self.natd # Dynamic port breakout command may wait for 60 second incase of breakout failure. - # So, to avoid socker red timeout, setting timeout to 120 seconds + # So, to avoid socket read timeout, setting timeout to 300 seconds + # Tries with 120 seconds, but still saw the failure, but 300 seconds consistently + # succeeded. Hence keeping it to 300 seconds while we dig deeper on why it + # take more than 60 seconds. self.client = docker.from_env(timeout=300) self.appldb = None