Attention is currently required from: iedemam, neels, laforge. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/27081 ) Change subject: stats: new trackers for lchan life duration ...................................................................... Patch Set 14: (1 comment) Patchset: PS14: First of all, my C compiler version (probably newer) fails to compile with this patch applied, it probably hints you in the error: """ /osmo-bsc/src/osmo-bsc/lchan_fsm.c:490:34: error: 'active_ms_rate_ctr' may be used uninitialized in this function [-Werror=maybe-uninitialized] 490 | lchan->active_ms = (struct osmo_time_cc){ | ~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~ 491 | .cfg = { | ~~~~~~~~ 492 | .gran_usec = 1*1000, | ~~~~~~~~~~~~~~~~~~~~ 493 | .forget_sum_usec = 1, | ~~~~~~~~~~~~~~~~~~~~~ 494 | .rate_ctr = active_ms_rate_ctr, | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 495 | .T_gran = -16, | ~~~~~~~~~~~~~~ 496 | .T_round_threshold = -17, | ~~~~~~~~~~~~~~~~~~~~~~~~~ 497 | .T_forget_sum = -18, | ~~~~~~~~~~~~~~~~~~~~ 498 | .T_defs = bts->network->T_defs, | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 499 | }, | ~~ 500 | }; | ~ """ If you want to run the tests locally: $ make check Then, you can see the output of the failing log under $buildir/tests/testsuite.dir/<number>/testsuite.log (or something like that, writing it by hard). Those are generated only for failing tests. Since I think one of the VTY tests is failing too, you can run one of them for instance: make -C tests/ vty-test VTY_TEST=power_ctrl.vty And if you add U=u to the end, the vty file will be updated, you can use then git diff to see the difference with expected output. -- To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/27081 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-bsc Gerrit-Branch: master Gerrit-Change-Id: I1b0670c47cb5e0b7776eda89d1e71545ba0e3347 Gerrit-Change-Number: 27081 Gerrit-PatchSet: 14 Gerrit-Owner: iedemam &lt;michael(a)kapsulate.com&gt; Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge &lt;laforge(a)osmocom.org&gt; Gerrit-Reviewer: neels &lt;nhofmeyr(a)sysmocom.de&gt; Gerrit-CC: pespin &lt;pespin(a)sysmocom.de&gt; Gerrit-Attention: iedemam &lt;michael(a)kapsulate.com&gt; Gerrit-Attention: neels &lt;nhofmeyr(a)sysmocom.de&gt; Gerrit-Attention: laforge &lt;laforge(a)osmocom.org&gt; Gerrit-Comment-Date: Fri, 04 Mar 2022 18:39:11 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment