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

Lower backend Profiler log message to Debug #651

Merged
merged 1 commit into from
Mar 23, 2023
Merged

Conversation

d0ugal
Copy link
Contributor

@d0ugal d0ugal commented Mar 23, 2023

This can help reduce the logging volume as it is very common and in most cases it wont be changed often.

What this PR does / why we need it:

The Profiler message is logged frequently and not needed in most cases. Reducing it to Debug feels more appropriate

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

This can help reduce the logging volume as it is very common and in most
cases it wont be changed often.
@d0ugal d0ugal requested a review from a team as a code owner March 23, 2023 10:06
@d0ugal d0ugal requested review from marefr and xnyo and removed request for a team March 23, 2023 10:06
@CLAassistant
Copy link

CLAassistant commented Mar 23, 2023

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@xnyo xnyo left a comment

Choose a reason for hiding this comment

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

LGTM!

@d0ugal d0ugal requested review from wbrowne and removed request for marefr March 23, 2023 10:22
@d0ugal d0ugal merged commit 0ee4592 into main Mar 23, 2023
@d0ugal d0ugal deleted the d0ugal-log-profilder-msg branch March 23, 2023 11:28
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.

None yet

4 participants