[asterisk-commits] testsuite: Increase the shutdown timeout time. (testsuite[master])

[SVN commits to the Asterisk project]

Anonymous Coward #1000019 has submitted this change and it was merged. ( https://gerrit.asterisk.org/4548 )

Change subject: testsuite: Increase the shutdown timeout time.
......................................................................


testsuite: Increase the shutdown timeout time.

The shutdown timeout is too short.  As a result we were forcibly killing
Asterisk before some events are generated which can contribute to bouncing
tests and false ref leak reports.

ASTERISK-26527

Change-Id: Ib1020223120176ec9890c7fdb6abb6855a33a6a1
---
M lib/python/asterisk/asterisk.py
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  George Joseph: Looks good to me, approved
  Scott Griepentrog: Looks good to me, but someone else must approve
  Joshua Colp: Verified



diff --git a/lib/python/asterisk/asterisk.py b/lib/python/asterisk/asterisk.py
index 994aa5d..fd4076a 100755
--- a/lib/python/asterisk/asterisk.py
+++ b/lib/python/asterisk/asterisk.py
@@ -632,7 +632,7 @@
         else:
             # Schedule a kill. If we don't gracefully shut down Asterisk, this
             # will ensure that the test is stopped.
-            sched_time = 200 if self.valgrind_enabled else 10
+            sched_time = 200 if self.valgrind_enabled else 45
             self._stop_cancel_tokens.append(reactor.callLater(sched_time,
                                             __send_kill))
 

-- 
To view, visit https://gerrit.asterisk.org/4548
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib1020223120176ec9890c7fdb6abb6855a33a6a1
Gerrit-PatchSet: 1
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Richard Mudgett <rmudgett at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
Gerrit-Reviewer: Scott Griepentrog <sgriepentrog at digium.com>