URL: https://github.com/SSSD/sssd/pull/567 Author: fidencio Title: #567: TESTS: Increase test_resp_idle_timeout* timeout Action: opened
PR body: """ As suggested by Sumit, let's increase the timeout in the test_resp_idle_timeout* as those are a little bit on the edge. Related: https://pagure.io/SSSD/sssd/issue/3633 Resolves: https://pagure.io/SSSD/sssd/issue/3730 Signed-off-by: Fabiano Fidêncio <fiden...@redhat.com> """ To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/567/head:pr567 git checkout pr567
From 495d2858c49b707daf465740b4e96ca89ef057c9 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fabiano=20Fid=C3=AAncio?= <fiden...@redhat.com> Date: Fri, 4 May 2018 11:31:41 +0200 Subject: [PATCH] TESTS: Increase test_resp_idle_timeout* timeout MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit As suggested by Sumit, let's increase the timeout in the test_resp_idle_timeout* as those are a little bit on the edge. Related: https://pagure.io/SSSD/sssd/issue/3633 Resolves: https://pagure.io/SSSD/sssd/issue/3730 Signed-off-by: Fabiano Fidêncio <fiden...@redhat.com> --- src/tests/intg/test_secrets.py | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/src/tests/intg/test_secrets.py b/src/tests/intg/test_secrets.py index a18abaefd..7979cd386 100644 --- a/src/tests/intg/test_secrets.py +++ b/src/tests/intg/test_secrets.py @@ -643,7 +643,9 @@ def test_resp_idle_timeout_shutdown_slow(setup_for_resp_timeout_test): # so it would tick at 30, 60 and 90 seconds and the responder_idle_timeout # uses a greater-than comparison, so the 60-seconds tick wouldn't yet # trigger the process' shutdown. - p.wait(timeout=90) + # 100s has been chosen in order to take a safer path when running our CI + # tests. + p.wait(timeout=100) assert p.is_running() is False @@ -678,7 +680,9 @@ def test_resp_idle_timeout_postpone_shutdown_slow(setup_for_resp_timeout_test, # updated). assert p.is_running() is True - # Wait more 60s in order to be sure that the process actually is shutdown - # when it should be. - p.wait(timeout=60) + # Wait at least 60s in order to be sure that the process actually is + # shutdown when it should be. + # 70s has been chosen in order to take a safer path when running our CI + # tests. + p.wait(timeout=70) assert p.is_running() is False
_______________________________________________ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
