Skip to content

Conversation

christarazi
Copy link
Member

This reverts commit c9f0def.

This introduced a bug into the code which can fatal Cilium whenever
there's a new pod and a policy update.

See #24021 for more details.

@christarazi christarazi added 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. labels Mar 10, 2023
@maintainer-s-little-helper
Copy link

Commit e735799651fcd70de3588c7e9e399c7783344a0c does not contain "Signed-off-by".

Please follow instructions provided in https://docs.cilium.io/en/stable/contributing/development/contributing_guide/#developer-s-certificate-of-origin

@maintainer-s-little-helper maintainer-s-little-helper bot added dont-merge/needs-sign-off The author needs to add signoff to their commits before merge. dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. and removed dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Mar 10, 2023
This reverts commit c9f0def.

This introduced a bug into the code which can fatal Cilium whenever
there's a new pod and a policy update.

See cilium#24021 for more details.

Signed-off-by: Chris Tarazi <chris@isovalent.com>
@christarazi christarazi force-pushed the pr/christarazi/revert-deny-stable branch from e735799 to 9f99b88 Compare March 10, 2023 02:22
@maintainer-s-little-helper
Copy link

Commit e735799651fcd70de3588c7e9e399c7783344a0c does not contain "Signed-off-by".

Please follow instructions provided in https://docs.cilium.io/en/stable/contributing/development/contributing_guide/#developer-s-certificate-of-origin

@christarazi
Copy link
Member Author

/test

@christarazi
Copy link
Member Author

Superseded by #24322

@christarazi christarazi deleted the pr/christarazi/revert-deny-stable branch March 13, 2023 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dont-merge/needs-sign-off The author needs to add signoff to their commits before 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.

1 participant