You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ozone.apache.org by GitBox <gi...@apache.org> on 2021/08/21 02:59:51 UTC
[GitHub] [ozone] jwminton opened a new pull request #2558: HDDS-5319 Intermittent failure in TestSCMUpdateServiceGrpcServer
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: issues-unsubscribe@ozone.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org
[GitHub] [ozone] jwminton commented on pull request #2558: HDDS-5319 Intermittent failure in TestSCMUpdateServiceGrpcServer
Posted by GitBox <gi...@apache.org>.
jwminton commented on pull request #2558:
URL: https://github.com/apache/ozone/pull/2558#issuecomment-1001878610
Yes, the shortest path will be for you to push that change if its not too
much trouble.
Thanks.
On Mon, Dec 27, 2021 at 3:57 AM Doroszlai, Attila ***@***.***>
wrote:
> @jwminton <https://github.com/jwminton> Thanks again for the patch. I
> would like to propose merging your improvements, but keeping the test
> disabled for now. If you agree, can you please push another commit to
> restore @Ignore, or let me know if I should make that change?
>
> —
> Reply to this email directly, view it on GitHub
> <https://github.com/apache/ozone/pull/2558#issuecomment-1001531652>, or
> unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AQJPZ4DZWTVRBQDPN5DZ3ODUTBH2FANCNFSM5CRLZPMQ>
> .
> You are receiving this because you were mentioned.Message ID:
> ***@***.***>
>
--
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: issues-unsubscribe@ozone.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org
[GitHub] [ozone] adoroszlai merged pull request #2558: HDDS-5319 Intermittent failure in TestSCMUpdateServiceGrpcServer
Posted by GitBox <gi...@apache.org>.
adoroszlai merged pull request #2558:
URL: https://github.com/apache/ozone/pull/2558
--
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: issues-unsubscribe@ozone.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org
[GitHub] [ozone] adoroszlai commented on pull request #2558: HDDS-5319 Intermittent failure in TestSCMUpdateServiceGrpcServer
Posted by GitBox <gi...@apache.org>.
adoroszlai commented on pull request #2558:
URL: https://github.com/apache/ozone/pull/2558#issuecomment-1001531652
@jwminton Thanks again for the patch. I would like to propose merging your improvements, but keeping the test disabled for now. If you agree, can you please push another commit to restore `@Ignore`, or let me know if I should make that change?
--
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: issues-unsubscribe@ozone.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org