[Asterisk-code-review] pjsip/subscriptions/ast restart: Sporadic failures due to NO... (testsuite[13])
Kevin Harwell
asteriskteam at digium.com
Thu Jun 28 11:51:49 CDT 2018
Kevin Harwell has uploaded this change for review. ( https://gerrit.asterisk.org/9307
Change subject: pjsip/subscriptions/ast_restart: Sporadic failures due to NOTIFY retransmission
......................................................................
pjsip/subscriptions/ast_restart: Sporadic failures due to NOTIFY retransmission
A race can sometimes occur between receiving a 200 OK for a NOTIFY and
retransmission of it, which will cause the test to fail if the NOTIFY gets
retransmitted.
This patch increases the t1 timer value in pjsip, so the test has time to
receive and process the NOTIFY's 200 OK without the retransmission timer being
fired first.
ASTERISK-27764 #close
Change-Id: I6bd5d7ddd9694140360392f70daba1b39c9c61d9
---
M tests/channels/pjsip/subscriptions/ast_restart/configs/ast1/pjsip.conf
1 file changed, 6 insertions(+), 0 deletions(-)
git pull ssh://gerrit.asterisk.org:29418/testsuite refs/changes/07/9307/1
diff --git a/tests/channels/pjsip/subscriptions/ast_restart/configs/ast1/pjsip.conf b/tests/channels/pjsip/subscriptions/ast_restart/configs/ast1/pjsip.conf
index ead6d27..4c43956 100644
--- a/tests/channels/pjsip/subscriptions/ast_restart/configs/ast1/pjsip.conf
+++ b/tests/channels/pjsip/subscriptions/ast_restart/configs/ast1/pjsip.conf
@@ -6,6 +6,12 @@
; the subscription recreation to be seen after the restart.
debug=yes
+[system]
+type=system
+; Sometimes we are still processing the 200 OK for a NOTIFY, and another
+; NOTIFY gets retransmitted. So raise the t1 timer a bit.
+timer_t1=2000
+
[local]
type=transport
protocol=udp
--
To view, visit https://gerrit.asterisk.org/9307
To unsubscribe, visit https://gerrit.asterisk.org/settings
Gerrit-Project: testsuite
Gerrit-Branch: 13
Gerrit-MessageType: newchange
Gerrit-Change-Id: I6bd5d7ddd9694140360392f70daba1b39c9c61d9
Gerrit-Change-Number: 9307
Gerrit-PatchSet: 1
Gerrit-Owner: Kevin Harwell <kharwell at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180628/86019324/attachment-0001.html>
More information about the asterisk-code-review
mailing list