-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
cancel_test doesn't tolerate redundant events #3366
Comments
Yeah, the |
@afrittoli Not sure from your comment what you think the remediation should be? |
For now I changed the specific test to allow for duplicate events, and I filed another issue about investigating this further #3375 |
Issues go stale after 90d of inactivity. /lifecycle stale Send feedback to tektoncd/plumbing. |
Stale issues rot after 30d of inactivity. /lifecycle rotten Send feedback to tektoncd/plumbing. |
Rotten issues close after 30d of inactivity. /close Send feedback to tektoncd/plumbing. |
@tekton-robot: Closing this issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Expected Behavior
Consistently passing test 😉
Actual Behavior
Intermittent flake with:
There are a number of reasons that this could be emitting multiple events of the same kind, including possibly stale informer caches during the Reconcile that emits the stale event.
I'd think the test should probably check "at least once" for each of the two events instead? cc @afrittoli
Steps to Reproduce the Problem
It's intermittent, so run it a lot 🤷
Additional Info
I'm running e2e on KinD on Github Actions, with Tekton at ~HEAD.
The text was updated successfully, but these errors were encountered: