-
Notifications
You must be signed in to change notification settings - Fork 579
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
tests: improve AdminOperationsFuzzer.wait #5993
Conversation
f"wait: terminating for error {self.error}") | ||
raise self.error | ||
|
||
if self.executed > count: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is the same as in the original, but I think this should be >=?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hmm, I guess it doesn't really matter as the test is waiting for an arbitrary number anyway, but yeah it's more intuitive to be >=, I'll update this.
This can drop out earlier when there is an error, rather than waiting in vain for the execution count to reach a target that it never will. Related: redpanda-data#5950
65ebaef
to
e0ff6b9
Compare
f"wait: terminating for error {self.error}") | ||
raise self.error | ||
|
||
if self.executed >= count: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the original predicate was self.executed > count
. not sure if it really matters
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, this was a bit of drive-by cleanup to make the thing actually run count
times (alexey's suggestion)
/backport v22.2.x |
Cover letter
This can drop out earlier when there is an error,
rather than waiting in vain for the execution count
to reach a target that it never will.
Related: #5950
Backport Required
UX changes
None
Release notes