Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove logging for messages not relevant to config-manager #161

Merged
merged 1 commit into from
Apr 12, 2024

Conversation

r14chandra
Copy link
Collaborator

Why do we need this change? 💭

Remove logging statements for messages not relevant to the service's processing.

Below are the logs for messages which are not processed by config-manager.

{"level":"debug","config":{"AppName":"config-manager","AWSAccessKeyId":"...","AWSRegion":"us-east-1","AWSSecretAccessKey":"....","CloudConnectorClientID":"config-manager","CloudConnectorHost":{"Value":{"Scheme":"http","Opaque":"","User":null,"Host":"cloud-connector.cloud-connector-stage.svc.cluster.local:8080","Path":"/api/cloud-connector/","RawPath":"","OmitHost":false,"ForceQuery":false,"RawQuery":"","Fragment":"","RawFragment":""},"Text":"http://cloud-connector.cloud-connector-stage.svc.cluster.local:8080/api/cloud-connector/"},"CloudConnectorPSK":"8b58ec79f59cba06576dfecb","CloudConnectorTimeout":10,"DBHost":"config-manager-stage.cm3mmfyi8atf.us-east-1.rds.amazonaws.com","DBName":"insights","DBPass":"...","DBPort":5432,"DBUser":"insights","DispatcherBatchSize":50,"DispatcherHost":{"Value":{"Scheme":"http","Opaque":"","User":null,"Host":"playbook-dispatcher-api.playbook-dispatcher-stage.svc.cluster.local:8000","Path":"","RawPath":"","...
{"level":"info","command":"inventory-consumer","time":"2024-04-12T04:52:47Z","caller":"/opt/app-root/src/internal/cmd/inventoryconsumer/cmd.go:26","message":"starting command"}
{"level":"trace","module":"inventory-consumer","offset":388381,"time":"2024-04-12T04:53:13Z","caller":"/opt/app-root/src/internal/cmd/inventoryconsumer/cmd.go:54","message":"handling kafka message"}
{"level":"trace","module":"inventory-consumer","offset":388382,"time":"2024-04-12T04:53:13Z","caller":"/opt/app-root/src/internal/cmd/inventoryconsumer/cmd.go:54","message":"handling kafka message"}
{"level":"trace","module":"inventory-consumer","offset":388383,"time":"2024-04-12T04:53:13Z","caller":"/opt/app-root/src/internal/cmd/inventoryconsumer/cmd.go:54","message":"handling kafka message"}

Documentation update? 📝

  • Yes
  • No

💂‍♂️ Checklist 🎯

  • Bugfix
  • New Feature
  • Refactor
  • Unittests Added
  • DRY code
  • Dependency Added
  • DB Migration Added

Additional 📣

Feel free to add any other relevant details such as links, notes, screenshots, here.

@r14chandra r14chandra merged commit 77d47dd into RedHatInsights:master Apr 12, 2024
4 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.

1 participant