[Asterisk-code-review] qualify timeout: Increase qualify timeout. (testsuite[master])

Mark Michelson asteriskteam at digium.com
Tue Nov 24 17:44:26 CST 2015


Mark Michelson has uploaded a new change for review.

  https://gerrit.asterisk.org/1715

Change subject: qualify_timeout: Increase qualify timeout.
......................................................................

qualify_timeout: Increase qualify timeout.

The qualify_timeout test fails under Jenkins test agents occasionally.
After investigating, this was because the qualify was timing out before
the AMI session was connected. Since the AMI session was not connected,
the required ConnectionStatus event would not get sent, meaning the test
would fail.

By increasing the qualify timeout, we can better ensure that the
qualify_timeout will happen after AMI has been connected, and we will
see the event we expect to see.

Change-Id: If9a011f432e30e406f6dace4c8e2712aedb593b8
---
M tests/channels/pjsip/qualify/qualify_timeout/configs/ast1/pjsip.conf
M tests/channels/pjsip/qualify/qualify_timeout/test-config.yaml
2 files changed, 4 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/testsuite refs/changes/15/1715/1

diff --git a/tests/channels/pjsip/qualify/qualify_timeout/configs/ast1/pjsip.conf b/tests/channels/pjsip/qualify/qualify_timeout/configs/ast1/pjsip.conf
index 14ad57e..3060582 100644
--- a/tests/channels/pjsip/qualify/qualify_timeout/configs/ast1/pjsip.conf
+++ b/tests/channels/pjsip/qualify/qualify_timeout/configs/ast1/pjsip.conf
@@ -15,5 +15,5 @@
 [sipp]
 type=aor
 contact=sip:127.0.0.1:5061
-qualify_frequency=6000
-qualify_timeout = 2
+qualify_frequency=10000
+qualify_timeout = 5
diff --git a/tests/channels/pjsip/qualify/qualify_timeout/test-config.yaml b/tests/channels/pjsip/qualify/qualify_timeout/test-config.yaml
index a5460d4..59a6372 100644
--- a/tests/channels/pjsip/qualify/qualify_timeout/test-config.yaml
+++ b/tests/channels/pjsip/qualify/qualify_timeout/test-config.yaml
@@ -21,11 +21,11 @@
 
 test-object-config:
     fail-on-any: False
-    reactor-timeout: 15
+    reactor-timeout: 25
     test-iterations:
         -
             scenarios:
-                - { 'key-args': {'scenario': 'options.xml', '-i': '127.0.0.1', '-d': '6000', '-p': '5061'} }
+                - { 'key-args': {'scenario': 'options.xml', '-i': '127.0.0.1', '-d': '10000', '-p': '5061'} }
 
 ami-config:
     -

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: If9a011f432e30e406f6dace4c8e2712aedb593b8
Gerrit-PatchSet: 1
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Mark Michelson <mmichelson at digium.com>



More information about the asterisk-code-review mailing list