Hi Andrew, > Could you push as a separate change the code that reliably gives you > the error in the LISP unit test
I tried but today, whatever I do, I cannot reproduce the test failure anymore. All tests pass now even when I try exactly the same code for which the test failed yesterday. For example, Patchset 4 for https://gerrit.fd.io/r/c/vpp/+/27280 failed yesterday, but now I created Patchset 8 which is identical to Patchset 4, and Patchset 8 passes all tests. I don't know, maybe something changed in the testing environment since yesterday, or maybe the issue was never reproducible, it was just a coincidence that made it seem that way yesterday. The good news is that the fix I wanted to do now passes the tests also when written as Ole suggested, with collector_port as u32 and a bounds check added: https://gerrit.fd.io/r/c/vpp/+/27280 It would be great if that could get merged. Best regards, Elias
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#16556): https://lists.fd.io/g/vpp-dev/message/16556 Mute This Topic: https://lists.fd.io/mt/74491544/21656 Group Owner: vpp-dev+ow...@lists.fd.io Unsubscribe: https://lists.fd.io/g/vpp-dev/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
