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

feat: Move contextless log messages to debug logging - Fixes #13918 #13919

Closed
wants to merge 1 commit into from

Conversation

kizzie
Copy link

@kizzie kizzie commented Nov 19, 2024

Before you open your PR

When you open your PR

  • PR title format should also conform to conventional commits.
  • "Fixes #" is in both the PR title (for release notes) and this description (to automatically link and close the issue).
  • Create the PR as draft.
  • Once builds are green, mark your PR "Ready for review".

When changes are requested, please address them and then dismiss the review to get it reviewed again.

-->

Fixes #13918

Motivation

'node changed' 'node unchanged' and 'task-result changed' flood the logs when running argo in production with many workflows and parallel runs, move these log statements to debug logging so that we can filter them

Modifications

Change logging for those three statements from Info to Debug

Verification

Running the tests

'node changed' 'node unchanged' and 'task-result changed' flood
the logs when running argo in production with many workflows and
parallel runs, move these log statements to debug logging so
that we can filter them
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.

Move some of the contextless log messages to debug level logging
1 participant