<p><a href="https://gerrit.osmocom.org/13392">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/13392/4//COMMIT_MSG">Commit Message:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/13392/4//COMMIT_MSG@33">Patch Set #4, Line 33:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">Note: at least osmo-msc's msc_vlr_tests' expected output needs to be adjusted<br>after merging this, because of logging changes for FSM deallocations<br></pre></blockquote></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">The main cause is that the order of deallocation changes. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">and actually also, what was a simple "Deallocated" logging before, may now become "Listing for deallocation with [other FSM inst id]", and there is s/"Terminating"/"Terminating in cascade".<br>see http://git.osmocom.org/osmo-msc/commit/?h=neels/ho&id=0a5bd401af092019cdf65aa1a1131cebefe96b76</p><p style="white-space: pre-wrap; word-wrap: break-word;">I think it is only mildly bad, the bad thing rather that this deallocation logging is in that osmo-msc test output in the first place. We've had a number of occasions where something affected that test output and adjusted it.</p><p style="white-space: pre-wrap; word-wrap: break-word;">But since I want to log state transitions, I also have to include deallocation logging, no way around it in the current FSM instance log category and level scheme (besides grepping output maybe)</p><p style="white-space: pre-wrap; word-wrap: break-word;">I have also considered a number of times to detach those tests from osmo-msc.git, or even lose them entirely; because they continuously introduce a bit of work like this on and off, and considerably blow up the git diffs, and also you don't like them ;) ... But in my reckoning the benefits still outweigh the drawbacks: it is often quite useful to track exactly how the logging in osmo-msc changes, which the ttcn3 tests don't (and can't) provide, and the fake time that ttcn3 can't provide makes for an ultra short dev cycle when hunting bugs. Minuses are complex hacks to make them work in a dry-run way, blown up git diffs, test failures from trivial logging changes. Am open to discussion there.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/13392">change 13392</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/13392"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmocore </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I8eda67540a1cd444491beb7856b9fcd0a3143b18 </div>
<div style="display:none"> Gerrit-Change-Number: 13392 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 02 Apr 2019 03:48:17 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>