<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://reviewboard.asterisk.org/r/2796/">https://reviewboard.asterisk.org/r/2796/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">The scenarios used for versions <= 11 (assuming they're the same ones that are in the current trunk of the testsuite) don't actually appear to use direct media. For instance, in phone_A.xml, after completing the initial INVITE transaction with Asterisk, the next expected event is the BYE to end the call. I would expect the same three re-invite transactions that phone_A_12.xml has (first to begin direct media, second to reinvite back to Asterisk for MOH, third to reinstate direct media). In fact, I wouldn't be surprised to see a fourth reinvite to direct the media back to Asterisk again when phone B sends its BYE to Asterisk.
Similarly the non-12 phone B scenarios don't appear to have the reinvites I would expect to see in order to initiate direct media.
I know the testsuite has required many tweaks to account for differences in Asterisk 12, but I would expect that the SIPp scenarios used for this test should be the same between different Asterisk versions. Here is what I suggest doing:
1) Using the phone_A.xml and any of the various non-12 phone B scenarios, manually run against Asterisk 11 and try to debug why it is that direct media reinvites do not get sent to the scenarios. If the reason is legitimate, then we likely should also not be offering direct media in 12 given the same setup. If Asterisk 11 *should* be offering direct media, then Astrisk 11 (and possibly 1.8) will need to be adjusted.
Based on the result of this, the path can branch a couple of different ways:
2a) If Asterisk 12 should not be offering direct media in this scenario, then the test will need to be adjusted in a way that will allow for direct media to be used. Once the test is altered properly, then the scenarios between Asterisk 11 and Asterisk 12 should be identical. If we find that the same scenarios do not pass in both 11 and 12, then we can look at the differences and try to rectify them.
2b) If Asterisk 11 should be offering direct media in this scenario, then Asterisk 11 should be altered to properly send the necessary reinvites. Like with 2a, I would expect that the same scenarios should then work with both Asterisk 11 and Asterisk 12. If not, then we need to find what is different and figure out how to resolve the issue.</pre>
<br />
<p>- Mark</p>
<br />
<p>On August 26th, 2013, 8:18 p.m. UTC, jrose wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.org/static/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Asterisk Developers, Joshua Colp, kmoore, Matt Jordan, and Mark Michelson.</div>
<div>By jrose.</div>
<p style="color: grey;"><i>Updated Aug. 26, 2013, 8:18 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="https://issues.asterisk.org/jira/browse/https://issues.asterisk.org/jira/browse/ASTERISK-22217">https://issues.asterisk.org/jira/browse/ASTERISK-22217</a>
</div>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
testsuite
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">On Friday I committed a patch which addressed some bugs with holding in Asterisk 12 while using native RTP bridges and directmedia. As part of that effort, the SIP hold tests in the testsuite were split up and divided into tests which used direct media and tests which didn't use direct media. At the time, Asterisk 12 failed the tests which used direct media. This patch fixes those tests by making the test use Asterisk 12 specific sipp scenarios (which were based on the existing scenarios). The main difference between the Asterisk 12 scenarios and their older counterparts was always just the addition of more expected invites and responses to those invites on account of how directmedia is established in Asterisk 12 both during the initial setup.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Ran modified tests against Asterisk 12.
Ran modified tests against Asterisk 11.
Repeated this process many times to make sure the results were consistent.
Tracked the invites in 12 against the code that was generating them.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>/asterisk/trunk/tests/channels/SIP/sip_hold_direct_media/sipp/phone_B_12_media_restrict.xml <span style="color: grey">(PRE-CREATION)</span></li>
<li>/asterisk/trunk/tests/channels/SIP/sip_hold_direct_media/sipp/phone_B_12_IP_restrict.xml <span style="color: grey">(PRE-CREATION)</span></li>
<li>/asterisk/trunk/tests/channels/SIP/sip_hold_direct_media/sipp/phone_B_12_IP_media_restrict.xml <span style="color: grey">(PRE-CREATION)</span></li>
<li>/asterisk/trunk/tests/channels/SIP/sip_hold_direct_media/sipp/phone_A_12_type2.xml <span style="color: grey">(PRE-CREATION)</span></li>
<li>/asterisk/trunk/tests/channels/SIP/sip_hold_direct_media/sipp/phone_A_12.xml <span style="color: grey">(PRE-CREATION)</span></li>
<li>/asterisk/trunk/tests/channels/SIP/sip_hold_direct_media/run-test <span style="color: grey">(4044)</span></li>
</ul>
<p><a href="https://reviewboard.asterisk.org/r/2796/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>