Skip to content
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

istioctl: add a flag to delete all waypoints in a namespace #47818

Merged
merged 4 commits into from
Nov 15, 2023

Conversation

hanxiaop
Copy link
Member

@hanxiaop hanxiaop commented Nov 10, 2023

Please provide a description of this PR:
Fix #47539 (comment)

@hanxiaop hanxiaop requested a review from a team as a code owner November 10, 2023 08:11
@istio-policy-bot
Copy link

🤔 🐛 You appear to be fixing a bug in Go code, yet your PR doesn't include updates to any test files. Did you forget to add a test?

Courtesy of your friendly test nag.

@istio-testing istio-testing added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Nov 10, 2023
@hanxiaop
Copy link
Member Author

I'll add this to the integration test once #47615 is merged.

@hanxiaop hanxiaop requested a review from a team as a code owner November 13, 2023 06:19
@hanxiaop
Copy link
Member Author

/test integ-security-multicluster

@hanxiaop
Copy link
Member Author

/test integ-security-multicluster

1 similar comment
@GregHanson
Copy link
Member

/test integ-security-multicluster

@hanxiaop
Copy link
Member Author

Interesting, where is the integ-telemetry-discovery_istio test going?

@hanxiaop
Copy link
Member Author

@istio/wg-test-and-release-maintainers PTAL, thanks

@hanxiaop hanxiaop requested a review from jwendell November 15, 2023 04:51
@jwendell
Copy link
Member

/retest

@istio-testing istio-testing merged commit 55d3690 into istio:master Nov 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/user experience size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants