jwminton opened a new pull request #2558: URL: https://github.com/apache/ozone/pull/2558
## What changes were proposed in this pull request? There are three failures reported in this. Two happen in testClientUpdateWithDelayedRevoke and one in testClientUpdateWithRestart. For the two failures that occur within testClientUpdateWithDelayedRevoke() the problem is that after revoking cert 5, after the waitFor, the next tests are trying to test pre-removal status of things while the cert is waiting to be revoked. The problem is there's nothing guaranteeing that these pre-revoke tests will happen before the timeout/revoke happens. Thus the two different failures. There are two tested values the revoke can affect and the specific failure depends on which operation the test is on when the revoke happens. Unless there's a way to guarantee this pre-revoke state but wihout the threat of being revoked, it may be best to let these two tests accommodate normal execution and the case where removal happened. Thus, the two asserts that are checking for range relations rather than for exact values. For the testClientUpdateWithRestart failure, mid-test the updateCount value isn't getting tracked correctly and its possible for the test execution to get ahead of what states the client/server should be in. The updateCount is tested for being 4 twice, missing an increment, guaranteeing that the last two waitFor's never wait because by definition, their predicates are already satisfied at the time of execution. ## What is the link to the Apache JIRA https://issues.apache.org/jira/browse/HDDS-5319 ## How was this patch tested? Manually, githubCI's, in my IDE and repeatedly in loops in bash -- 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]
