8000 [v1.16] docs: Add note for CNP empty slices semantic under v1.16 section by pippolo84 · Pull Request #34008 · cilium/cilium · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

[v1.16] docs: Add note for CNP empty slices semantic under v1.16 section #34008

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

Merged

Conversation

pippolo84
Copy link
Member

The semantic change for empty non-nil slice in CNPs targeted v1.16, but the related release note was inadvertently appended under the "v1.15 Upgrade Notes" section (see PR #29608). Therefore, that note was deleted when preparing the documentation for v1.16 in #33703

The PR adds back the upgrade note in the v1.16 section where it belongs.

Related: e47e295 ("docs: cleanup upgrade docs on 1.16")
Related: 966757d ("docs: add upgrade note for dangling cidrGroupRefs")
Fixes: 5f77d50 ("docs: Add upgrade note for CNP empty slices new semantic")

Following the change in the semantic of an empty non-nil slice in CNPs,
an upgrade note is added to the guide for v1.16.

The semantic change targeted v1.16 but this release note was
inadvertently appended under the "v1.15 Upgrade Notes" section and thus
deleted when preparing the documentation for v1.16.

Related: e47e295 ("docs: cleanup upgrade docs on 1.16")
Related: 966757d ("docs: add upgrade note for dangling cidrGroupRefs")
Fixes: 5f77d50 ("docs: Add upgrade note for CNP empty slices new semantic")

Signed-off-by: Fabio Falzoi <fabio.falzoi@isovalent.com>
@pippolo84 pippolo84 added area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. sig/policy Impacts whether traffic is allowed or denied based on user-defined policies. release-note/misc This PR makes changes that have no direct user impact. affects/v1.16 This issue affects v1.16 branch labels Jul 25, 2024
@pippolo84 pippolo84 requested a review from a team as a code owner July 25, 2024 09:02
@maintainer-s-little-helper maintainer-s-little-helper bot added backport/1.16 This PR represents a backport for Cilium 1.16.x of a PR that was merged to main. kind/backports This PR provides functionality previously merged into master. labels Jul 25, 2024
@pippolo84 pippolo84 removed kind/backports This PR provides functionality previously merged into master. backport/1.16 This PR represents a backport for Cilium 1.16.x of a PR that was merged to main. affects/v1.16 This issue affects v1.16 branch labels Jul 25, 2024
@julianwiedmann
Copy link
Member

/test-backport-1.16

@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Jul 25, 2024
@squeed squeed added this pull request to the merge queue Jul 25, 2024
Merged via the queue into cilium:v1.16 with commit 83a81ce Jul 25, 2024
63 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact. sig/policy Impacts whether traffic is allowed or denied based on user-defined policies.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0