Skip to content

Log policy updates at debug level#43231

Merged
aanm merged 1 commit intocilium:mainfrom
sjohnsonpal:log-policy-updates-at-debug
Dec 16, 2025
Merged

Log policy updates at debug level#43231
aanm merged 1 commit intocilium:mainfrom
sjohnsonpal:log-policy-updates-at-debug

Conversation

@sjohnsonpal
Copy link
Copy Markdown
Contributor

High churn clusters log significantly more (10x to 20x) log lines since 1.17 at 783465b "policy: add policy import cell".

This commit changes the policy update log lines added in that commit to Debug level to avoid filling up the logs with policy update information by default.

@sjohnsonpal sjohnsonpal requested a review from a team as a code owner December 9, 2025 21:48
@sjohnsonpal sjohnsonpal requested a review from fristonio December 9, 2025 21:48
@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 Dec 9, 2025
@github-actions github-actions bot added sig/policy Impacts whether traffic is allowed or denied based on user-defined policies. kind/community-contribution This was a contribution made by a community member. labels Dec 9, 2025
@odinuge
Copy link
Copy Markdown
Member

odinuge commented Dec 10, 2025

/test

@odinuge
Copy link
Copy Markdown
Member

odinuge commented Dec 10, 2025

/ci-integration

Copy link
Copy Markdown
Member

@fristonio fristonio left a comment

Choose a reason for hiding this comment

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

LGTM, apart from a minor comment.

@squeed squeed added the release-note/misc This PR makes changes that have no direct user impact. label Dec 11, 2025
@maintainer-s-little-helper maintainer-s-little-helper bot added ready-to-merge This PR has passed all tests and received consensus from code owners to merge. and removed dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Dec 11, 2025
High churn clusters log significantly more (10x to 20x) log lines since 1.17
at 783465b "policy: add policy import cell".

This commit changes the policy update log lines added in that commit to
Debug level to avoid filling up the logs with policy update information
by default.

Signed-off-by: Steven Johnson <[email protected]>
@aanm aanm force-pushed the log-policy-updates-at-debug branch from 9889102 to 73e3f78 Compare December 16, 2025 09:54
@aanm aanm removed the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Dec 16, 2025
@aanm aanm enabled auto-merge December 16, 2025 09:54
@aanm
Copy link
Copy Markdown
Member

aanm commented Dec 16, 2025

/test

@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 Dec 16, 2025
@aanm aanm added this pull request to the merge queue Dec 16, 2025
Merged via the queue into cilium:main with commit c1eece7 Dec 16, 2025
76 checks passed
@cilium-release-bot cilium-release-bot bot moved this to Released in cilium v1.19.0 Feb 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/community-contribution This was a contribution made by a community member. 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

No open projects
Status: Released

Development

Successfully merging this pull request may close these issues.

5 participants