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

ingress: Rename LB annotation to annotation prefixes #21222

Merged
merged 1 commit into from
Sep 9, 2022

Conversation

sayboras
Copy link
Member

@sayboras sayboras commented Sep 7, 2022

This is to avoid any potential confusion from user. Also, related docs are updated.

Related: #20860

Signed-off-by: Tam Mach tam.mach@cilium.io

@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Sep 7, 2022
@sayboras sayboras added the release-note/minor This PR changes functionality that users may find relevant to operating Cilium. label Sep 7, 2022
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Sep 7, 2022
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.12.2 Sep 7, 2022
@sayboras sayboras force-pushed the tam/ingres-annotation-prefixes branch from a7fd57d to f735c57 Compare September 7, 2022 09:38
This is to avoid any potential confusion from user. Also, related
docs are updated.

Related: cilium#20860

Signed-off-by: Tam Mach <tam.mach@cilium.io>
@sayboras sayboras force-pushed the tam/ingres-annotation-prefixes branch from f735c57 to 9e2bf33 Compare September 7, 2022 09:47
@maintainer-s-little-helper maintainer-s-little-helper bot removed this from Needs backport from master in 1.12.2 Sep 7, 2022
@sayboras sayboras marked this pull request as ready for review September 7, 2022 09:48
@sayboras sayboras requested review from a team as code owners September 7, 2022 09:48
Copy link
Member

@qmonnet qmonnet left a comment

Choose a reason for hiding this comment

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

Looks good, thanks.
Given that the Helm value has changed name, does it deserve a mention in the Upgrade Notes (Documentation/operations/upgrade.rst)?

@sayboras
Copy link
Member Author

sayboras commented Sep 7, 2022

Looks good, thanks. Given that the Helm value has changed name, does it deserve a mention in the Upgrade Notes (Documentation/operations/upgrade.rst)?

Right now, the original annotation changes are only available in master and 1.13.0-rc0, and this is a newly added feature, I don't think we need to highlight it in upgrade docs.

@sayboras
Copy link
Member Author

sayboras commented Sep 8, 2022

/test

@sayboras
Copy link
Member Author

sayboras commented Sep 9, 2022

Tests are all passed, most of the reviews are in. Marking this ready to merge.

In case of there is any other comments coming later, I will address in subsequent PRs.

@sayboras sayboras added ready-to-merge This PR has passed all tests and received consensus from code owners to merge. needs-backport/1.12 labels Sep 9, 2022
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.12.2 Sep 9, 2022
@maintainer-s-little-helper maintainer-s-little-helper bot removed the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Sep 9, 2022
@nebril nebril merged commit 481493b into cilium:master Sep 9, 2022
@sayboras sayboras deleted the tam/ingres-annotation-prefixes branch September 9, 2022 11:34
@nebril nebril added backport-done/1.12 The backport for Cilium 1.12.x for this PR is done. and removed needs-backport/1.12 labels Sep 13, 2022
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport done to v1.12 in 1.12.2 Sep 13, 2022
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport done to v1.12 in 1.12.2 Sep 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-done/1.12 The backport for Cilium 1.12.x for this PR is done. release-note/minor This PR changes functionality that users may find relevant to operating Cilium.
Projects
No open projects
1.12.2
Backport done to v1.12
Development

Successfully merging this pull request may close these issues.

None yet

5 participants