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

Fix webhooks created by tag set being deleted after re-installation #48780

Merged
merged 2 commits into from
Jan 16, 2024

Conversation

hanxiaop
Copy link
Member

@hanxiaop hanxiaop commented Jan 12, 2024

Please provide a description of this PR:
Fix #47423

@istio-testing
Copy link
Collaborator

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@istio-testing istio-testing added the do-not-merge/work-in-progress Block merging of a PR because it isn't ready yet. label Jan 12, 2024
@istio-testing istio-testing added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Jan 12, 2024
@hanxiaop hanxiaop added cherrypick/release-1.19 Set this label on a PR to auto-merge it to the release-1.19 branch cherrypick/release-1.20 Set this label on a PR to auto-merge it to the release-1.20 branch and removed size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Jan 12, 2024
@istio-testing istio-testing added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Jan 12, 2024
@hanxiaop hanxiaop marked this pull request as ready for review January 12, 2024 07:52
@hanxiaop hanxiaop requested a review from a team as a code owner January 12, 2024 07:52
@istio-testing istio-testing removed the do-not-merge/work-in-progress Block merging of a PR because it isn't ready yet. label Jan 12, 2024
area: installation
issue:
- 47423
releaseNotes:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: I'm a little worried that people may be confused because the wording and issue number on this are identical to the other #47538 which has already been included in https://istio.io/latest/news/releases/1.20.x/announcing-1.20.1/

maybe I'm over thinking it, or that people pay too close attention to our release notes.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@GregHanson Yes, this may be a problem, but I'm not sure which option is the best since the old patch didn't fix the problem well. Alternatively, I could take a close eye on the new patch release and remove the note in 1.20.1 on the doc site when 1.20.3 is released, although maybe we didn't do this before,. I'm the RM for both 1.19 and 1.20, and the note should be put in the right place instead of dups.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm OK with it going in as-is then if you're willing to adjust prior release notes

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

Copy link
Member

@GregHanson GregHanson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@istio-testing istio-testing merged commit 40ddb02 into istio:master Jan 16, 2024
28 checks passed
@istio-testing
Copy link
Collaborator

In response to a cherrypick label: new pull request created: #48820

@istio-testing
Copy link
Collaborator

In response to a cherrypick label: new pull request created: #48821

liwenhao0810 pushed a commit to liwenhao0810/istio that referenced this pull request Feb 1, 2024
…istio#48780)

* fix user created tag being deleted after re-installation

* add releasenotes
thedebugger pushed a commit to thedebugger/istio that referenced this pull request Mar 5, 2024
…istio#48780)

* fix user created tag being deleted after re-installation

* add releasenotes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/user experience cherrypick/release-1.19 Set this label on a PR to auto-merge it to the release-1.19 branch cherrypick/release-1.20 Set this label on a PR to auto-merge it to the release-1.20 branch size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

istio-operator/istioctl > 1.19 unexpectedly remove MutatingWebhookConfiguration::istio-revision-tag
4 participants