<p>Corey Farrell <strong>uploaded patch set #4</strong> to this change.</p><p><a href="https://gerrit.asterisk.org/6845">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">Add support for delayed shutdown of Asterisk when checking for leaks.<br><br>Add memcheck-delay-stop option to test-config.yaml in the test-object<br>config section.  This causes a pause during shutdown if Asterisk is run<br>under valgrind or has REF_DEBUG enabled.  This delay is not meant to<br>compensate for slowness caused by memory debugging, but to allow SIP<br>timers to clean resources before shutdown of Asterisk is initiated.<br><br>If set this options forces reactor-timeout to be at least 3 seconds<br>longer.  This is only done to stop the testsuite from printing a reactor<br>timeout warning for tests with a large memcheck-delay-stop value.<br><br>I had considered setting a non-zero default for memcheck-delay-stop but<br>the majority of tests do not require any delay.  Since the testsuite<br>contains nearly 1000 tests using a 7 second default delay would add<br>nearly 2 hours to the total runtime.  I estimate that only 20% of the<br>tests require any delay.  The amount of delay needed varies based on<br>which timers we need to wait for: timer_b, session expiration, defered<br>session termination.  The minimums for these timers range from 6.4 - 90<br>seconds.  In most cases using the minimum delay will require modification<br>of the test.  For example reconfiguring timer_t1 and timer_b in some basic<br>PJSIP call tests or reducing the 'Expires:' header in SIP SUBSCRIBE tests.<br><br>As an example of usage tests/channels/pjsip/accountcode has been<br>updated.  The timer_t1 and timer_b have been decreased to minimize the<br>delay needed.<br><br>ASTERISK-27306<br><br>Change-Id: I67ec89e4a116526ad34796a233e15fec794cf3e1<br>---<br>M lib/python/asterisk/apptest.py<br>M lib/python/asterisk/ari.py<br>M lib/python/asterisk/asterisk.py<br>M lib/python/asterisk/sipp.py<br>M lib/python/asterisk/test_case.py<br>M tests/channels/pjsip/accountcode/configs/ast1/pjsip.conf<br>M tests/channels/pjsip/accountcode/test-config.yaml<br>7 files changed, 48 insertions(+), 22 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/testsuite refs/changes/45/6845/4</pre><p>To view, visit <a href="https://gerrit.asterisk.org/6845">change 6845</a>. To unsubscribe, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/6845"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: testsuite </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>
<div style="display:none"> Gerrit-Change-Id: I67ec89e4a116526ad34796a233e15fec794cf3e1 </div>
<div style="display:none"> Gerrit-Change-Number: 6845 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>