10000 [release-1.26] CNI Pod delete edge case by istio-testing · Pull Request #56764 · istio/istio · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

[release-1.26] CNI Pod delete edge case #56764

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 3 commits into from
Closed

[release-1.26] CNI Pod delete edge case #56764

wants to merge 3 commits into from

Conversation

istio-testing
Copy link
Collaborator

This is an automated cherry-pick of #56738

ilrudie added 3 commits June 24, 2025 16:48
@ilrudie @istio-testing
don't leave a dangling UID in the snapshot if we fail with a non-retr…
dc811dc 
…yable

Signed-off-by: Ian Rudie <ian.rudie@solo.io>
@ilrudie @istio-testing
handle delete edge case
b4dd70c 
Signed-off-by: Ian Rudie <ian.rudie@solo.io>
@ilrudie @istio-testing
release note
33f6fab 
Signed-off-by: Ian Rudie <ian.rudie@solo.io>
@istio-testing istio-testing requested a review from a team as a code owner June 24, 2025 16:48
@istio-testing istio-testing added the release-notes-none Indicates a PR that does not require release notes. label Jun 24, 2025
@istio-testing istio-testing mentioned this pull request Jun 24, 2025
Merged
@istio-testing istio-testing added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Jun 24, 2025
@istio-testing
Copy link
Collaborator Author

@istio-testing: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
windowsbuildcheck_istio_release-1.26 33f6fab link true /test windowsbuildcheck
release-test_istio_release-1.26 33f6fab link true /test release-test
integ-basic-arm64_istio_release-1.26 33f6fab link true /test integ-basic-arm64
integ-ipv6_istio_release-1.26 33f6fab link true /test integ-ipv6
integ-helm_istio_release-1.26 33f6fab link true /test integ-helm
integ-pilot-istiodremote_istio_release-1.26 33f6fab link true /test integ-pilot-istiodremote
integ-security_istio_release-1.26 33f6fab link true /test integ-security
integ-pilot-multicluster_istio_release-1.26 33f6fab link true /test integ-pilot-multicluster
integ-pilot-istiodremote-mc_istio_release-1.26 33f6fab link true /test integ-pilot-istiodremote-mc
lint_istio_release-1.26 33f6fab link true /test lint
integ-distroless_istio_release-1.26 33f6fab link true /test integ-distroless
integ-ambient-ipv6_istio_release-1.26 33f6fab link true /test integ-ambient-ipv6
integ-ambient_istio_release-1.26 33f6fab link true /test integ-ambient
integ-ds_istio_release-1.26 33f6fab link true /test integ-ds
integ-pilot_istio_release-1.26 33f6fab link true /test integ-pilot
integ-security-istiodremote_istio_release-1.26 33f6fab link true /test integ-security-istiodremote
integ-telemetry_istio_release-1.26 33f6fab link true /test integ-telemetry
unit-tests-arm64_istio_release-1.26 33f6fab link true /test unit-tests-arm64
integ-telemetry-discovery_istio_release-1.26 33f6fab link true /test integ-telemetry-discovery
integ-telemetry-istiodremote_istio_release-1.26 33f6fab link true /test integ-telemetry-istiodremote
integ-telemetry-mc_istio_release-1.26 33f6fab link true /test integ-telemetry-mc
integ-cni_istio_release-1.26 33f6fab link true /test integ-cni
integ-security-multicluster_istio_release-1.26 33f6fab link true /test integ-security-multicluster
unit-tests_istio_release-1.26 33f6fab link true /test unit-tests
gencheck_istio_release-1.26 33f6fab link true /test gencheck

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-sigs/prow repository. I understand the commands that are listed here.

@ilrudie ilrudie mentioned this pull request Jun 24, 2025
Open
@ilrudie
Copy link
Contributor
ilrudie commented Jun 24, 2025

needed some manual fixing, new PR filed to replace this one

@ilrudie ilrudie closed this Jun 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
release-notes-none Indicates a PR that does not require release notes. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@istio-testing @ilrudie
0