<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 2:</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 2:</p><p style="white-space: pre-wrap; word-wrap: break-word;">I would avoid this kind of parameters in the future if there's no good reason for it other than making other tests fail/crash.<br>AFAIU, in this case running the new tests on older versions would simply make the newly added test fail, but wouldn't affect others.<br>It's fine letting the testset run for a few more seconds rather than now having to add parameters, later having to remove them, etc.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I agree that these parameters don't need to be added only to prevent a new test from failing on the latest release (vs nightly).</p><p style="white-space: pre-wrap; word-wrap: break-word;">Here the reason was different, this was needed or else a whole bunch of existing tests would fail on latest: https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/25185</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Oh, I've confused it with another variable.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Anyway, this variable was added, so existing tests TC_paging_ps_from_sgsn_* could be extended without having them starting to fail on latest. So IMHO it's still worth to add the variable for that, it's about making sure existing tests still pass on latest.</p><p style="white-space: pre-wrap; word-wrap: break-word;">https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/25157</p><p><a href="https://gerrit.osmocom.org/c/docker-playground/+/25156">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/docker-playground/+/25156">change 25156</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.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.osmocom.org/c/docker-playground/+/25156"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: docker-playground </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Iabd0cf658f2b8e2913824b899c5bd012c587c21d </div>
<div style="display:none"> Gerrit-Change-Number: 25156 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: daniel <dwillmann@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-Reviewer: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 23 Aug 2021 13:20:24 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>