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
Make watch order conformance test reliable #101950
Conversation
/retest |
test failure is #101953, unrelated /retest |
case event := <-watch.ResultChan(): | ||
case event, ok := <-watch.ResultChan(): | ||
if !ok { | ||
framework.Failf("Watch closed unexpectedly") |
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.
eventually we should change this to be retriable (conformance should pass if watches close) but i won’t make you do that here (i may end up having to do it)
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.
This is now a retry watcher (watchtools.NewRetryWatcher
), so it is already retrying/re-establishing the watch internally... we only get to this point if re-establishing the watch failed. Still possible, but more robust than it was before.
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.
Nice!
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: liggitt, smarterclayton The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/priority important-soon |
/retest Review the full test history for this PR. Silence the bot with an |
opened picks to 1.19, 1.20, 1.21 |
…950-upstream-release-1.21 Automated cherry pick of #101950: Make watch order conformance test reliable
…950-upstream-release-1.20 Automated cherry pick of #101950: Make watch order conformance test reliable
…950-upstream-release-1.19 Automated cherry pick of #101950: Make watch order conformance test reliable
What type of PR is this?
/kind bug
/kind failing-test
What this PR does / why we need it:
Fixes #101949