devmadhuu opened a new pull request, #5828: URL: https://github.com/apache/ozone/pull/5828
This PR fixes the Intermittent failure in TestStorageContainerManager#testContainerReportQueueTakingMoreTime. Intermittent failure was detected in below assertion: Assert.assertTrue(containerReportExecutors.longWaitInQueueEvents() >= 1); This assertion statement asserts that number of container reports, waiting in event queue more than queueWaitThreshold is greater than 1. This metric of longWaitInQueue is incremented if report is waiting in event queue longer than wait threshold value of 1s set by test case. Due to timing issue, assertion use to fail intermittently and this PR fixes that with usage of semaphore. This PR also improves the test case by reducing the time needed to finish the test case. ## What is the link to the Apache JIRA https://issues.apache.org/jira/browse/HDDS-8470 ## How was this patch tested? This patch is tested by running all test cases of TestStorageContainerManager in repeated CI flakytest workflow by running 500 iterations. Here is the green CI [link](https://github.com/devmadhuu/ozone/actions/runs/7244420268/job/19775989967) for the same. Flaky Workflow CI run passes for TestStorageContainerManager#testContainerReportQueueTakingMoreTime, failures are related to timeout for TestStorageContainerManager#testBlockDeletionTransactions for which we already have [HDDS-8492](https://issues.apache.org/jira/browse/HDDS-8492) -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
