gemmellr commented on PR #4584: URL: https://github.com/apache/activemq-artemis/pull/4584#issuecomment-1690242465
I wasnt able to make MQTTSecurityManagerTest.testSecurityManagerModifyClientIDAndStealConnection fail locally. Justin was able to get it to fail once while putting his system under outside stress during the run. I have seen 8 consecutive failures of the test during full test suite runs of either this PR branch or the original PR branch in a CI env I use. IT has been passing on all the other test runs being done from main etc. I made a further change to have Caffeine use an inline executor for its cleanup etc, and the test then passed on the first run. I'm going to do some more runs on it, but at the moment it does seem like the async cleanup is the likely reason for the test failure. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
