Skip to content

Conversation

viktor-kurchenko
Copy link
Contributor

@viktor-kurchenko viktor-kurchenko commented Jun 26, 2025

Once this PR is merged, a GitHub action will update the labels of these PRs:

 40217

[ upstream commit cb9bcd8 ]

In production clusters with several secrets, these messages may flood
the operator logs since they are also logged in case of no changes
(action=ignored). Example from a recent sysdump:

    $ grep "Reconciling secret" logs-cilium-operator-<redacted>-cilium-operator-20250626-083319.log | wc -l
    36789
    $ grep "Successfully reconciled Secret.*action=ignored" logs-cilium-operator-<redacted>-cilium-operator-20250626-083319.log | wc -l
36789

Decrease the log level to debug so the messages still appear in CI and
during development, but their value in production clusters is limited.

[ Backporter's notes:
  - resolved conflicts in: operator/pkg/secretsync/secretsync_reconcile.go
]

Signed-off-by: Tobias Klauser <tobias@cilium.io>
Signed-off-by: viktor-kurchenko <viktor.kurchenko@isovalent.com>
@viktor-kurchenko viktor-kurchenko added 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. labels Jun 26, 2025
@viktor-kurchenko viktor-kurchenko marked this pull request as ready for review June 26, 2025 12:22
@viktor-kurchenko viktor-kurchenko requested a review from a team as a code owner June 26, 2025 12:22
@viktor-kurchenko
Copy link
Contributor Author

/test

@tklauser
Copy link
Member

Thanks Viktor!

@tklauser tklauser enabled auto-merge June 26, 2025 12:23
@tklauser tklauser added this pull request to the merge queue Jun 26, 2025
@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 Jun 26, 2025
Merged via the queue into v1.16 with commit 66851a6 Jun 26, 2025
282 of 286 checks passed
@tklauser tklauser deleted the pr/v1.16-backport-2025-06-26-12-41 branch June 26, 2025 13:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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. ready-to-merge This PR has passed all tests and received consensus from code owners to merge.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants