Skip to content

Operator logs to logfmt and more expressive logs#1

Merged
gcaracuel merged 4 commits into
mainfrom
chore/improved_logging_format
Aug 26, 2025
Merged

Operator logs to logfmt and more expressive logs#1
gcaracuel merged 4 commits into
mainfrom
chore/improved_logging_format

Conversation

@gcaracuel

Copy link
Copy Markdown
Collaborator

No description provided.

@gcaracuel gcaracuel self-assigned this Aug 25, 2025
Copilot AI review requested due to automatic review settings August 26, 2025 07:07

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds structured logging to the SecretReconciler by converting log statements to logfmt format with key-value pairs for better observability. The changes focus on enriching log messages with contextual information like resource names and namespaces.

  • Added structured logging with key-value pairs to improve log searchability and monitoring
  • Enhanced error and info log messages with resource identifiers and namespace context
  • Removed an extra blank line for code formatting consistency

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread internal/controller/secret_controller.go Outdated
Comment thread internal/controller/secret_controller.go Outdated
Comment thread internal/controller/secret_controller.go Outdated
Comment thread internal/controller/secret_controller.go Outdated
Comment thread internal/controller/secret_controller.go Outdated
Comment thread internal/controller/secret_controller.go Outdated
Comment thread internal/controller/secret_controller.go Outdated
Comment thread internal/controller/secret_controller.go Outdated
Comment thread internal/controller/secret_controller.go Outdated
Comment thread internal/controller/secret_controller.go Outdated
@gcaracuel gcaracuel merged commit b2b65b2 into main Aug 26, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants