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
Update ingress conformance test for finalizers #96742
Update ingress conformance test for finalizers #96742
Conversation
Hi @spencerhance. Thanks for your PR. I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. 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. |
/assign @liggitt |
3412c45
to
f8d4ae5
Compare
f8d4ae5
to
c1f9dec
Compare
/ok-to-test |
/priority important-soon |
cd6ced0
to
b7c701a
Compare
b7c701a
to
78df51a
Compare
78df51a
to
47ea73b
Compare
/lgtm |
/lgtm |
/retest |
@liggitt looks like it just needs a milestone now |
@kubernetes/sig-release - would like a ruling on this for 1.20 vs 1.20.1. This should be merged and picked to 1.19 as well. Without this PR, this conformance test (added in 1.19) fails on clusters that use finalizers on ingress objects to manage releasing load balancer resources. |
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.
/lgtm
/milestone v1.20
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bowei, hasheddan, spencerhance 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 |
Thanks. Picked to 1.19 in #96953 |
/retest Review the full test history for this PR. Silence the bot with an |
1 similar comment
/retest Review the full test history for this PR. Silence the bot with an |
…2-upstream-release-1.19 Automated cherry pick of #96742: Update ingress conformance test for finalizers
What type of PR is this?
/kind failing-test
What this PR does / why we need it:
Updates Ingress Conformance tests to check for deletionTimestamp instead of 404 due to finalizers.
This will need to be cherry-picked to release-1.19