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
Revert "policy: Make selectorcache callbacks lock-free" #16769
Conversation
This reverts commit a75599d. has it seems to be causing a lot of FQDN flakes through the entire CI. Signed-off-by: André Martins <andre@cilium.io>
test-me-please |
Previous jenkins failures: https://jenkins.cilium.io/job/Cilium-PR-K8s-1.21-kernel-4.9/967/ |
test-me-please |
All Jenkins jobs were green |
test-me-please |
All Jenkins jobs were green |
test-me-please |
Previous jenkins failures: https://jenkins.cilium.io/job/Cilium-PR-K8s-1.19-kernel-5.4/577/ |
test-me-please |
All Jenkins jobs were green |
test-me-please |
All Jenkins jobs were green |
test-me-please |
Thanks for checking 🙏 Not sure why I thought #16529 was merged on the 24th :-/
I'd rather go the other way around: leave open and close if we see it's not happening anymore. That way contributors are less likely to open a new issue for the same flake (which would mean we'd lose track of previous information unless someone notices the duplicate). I also don't see a benefit to closing the issue; it's not blocking anything. |
PR #16529 was never backported to v1.8, removing the needs-backport/1.8 since you can't revert something that was never in the tree. |
This reverts commit a75599d.
has it seems to be causing a lot of FQDN flakes through the entire CI.
Signed-off-by: André Martins andre@cilium.io
Fixes: #16741
Fixes: #16724
Fixes: #16713
Fixes: #16659