Skip to content

Conversation

fristonio
Copy link
Member

Follow up fix for - 6380e52#diff-2a82969e4493f485b248bb9e4f9e1487c713053ab50fac9a0f7784ede78bf4b7R667-R674
See commit message for more details.

Fixes: #36493

This commit is a follow up fix for:
6380e52 ("pkg/endpoint: reduce missed-policy-update severity when safe")

When regenerating restored endpoints the state of endpoint is marked as
`StateRestoring` instead of `StateWaitingToRegenerate`. This commit
extends the above fix to skip logging a warning when policy revision
update is missed for restoring endpoints.

Signed-off-by: Deepesh Pathak <deepesh.pathak@isovalent.com>
@fristonio fristonio requested a review from a team as a code owner August 12, 2025 20:20
@fristonio fristonio added release-note/misc This PR makes changes that have no direct user impact. area/CI Continuous Integration testing issue or flake sig/policy Impacts whether traffic is allowed or denied based on user-defined policies. labels Aug 12, 2025
@fristonio
Copy link
Member Author

fristonio commented Aug 13, 2025

/test
ci-integration: #41093
K8s NetworkPolicy E2E tests - #39968

@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 Aug 13, 2025
@fristonio fristonio added this pull request to the merge queue Aug 13, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 13, 2025
@fristonio fristonio added this pull request to the merge queue Aug 13, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 13, 2025
@fristonio fristonio added this pull request to the merge queue Aug 13, 2025
Merged via the queue into cilium:main with commit 075b346 Aug 13, 2025
74 of 77 checks passed
@fristonio fristonio deleted the pr/fristonio/fix/ci/policy-regen-warning branch August 13, 2025 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/CI Continuous Integration testing issue or flake 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.

CI: Cilium E2E Upgrade: Endpoint missed a policy revision; triggering regeneration
3 participants