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

Fixes #38150 - Switch log level of 2 harmless messages to debug level #11287

Merged
merged 1 commit into from
Jan 27, 2025

Conversation

hao-yu
Copy link
Contributor

@hao-yu hao-yu commented Jan 23, 2025

No description provided.

Copy link
Member

@ianballou ianballou left a comment

Choose a reason for hiding this comment

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

This log + the logic that chooses the last version_href have been around for such a long time now that I don't think it's necessary to keep close track of them anymore.

What do you think about changing it to a debug log? Reducing the amount of normal logs is always helpful where possible.

If a user is having issues that are difficult to debug, I'd hope that enabling debug logging is a normal first step.

@hao-yu hao-yu force-pushed the fix_warning_log_to_info_log branch from fe1c3ab to e5d031b Compare January 27, 2025 00:56
@hao-yu hao-yu force-pushed the fix_warning_log_to_info_log branch from e5d031b to eda8a83 Compare January 27, 2025 00:57
@hao-yu hao-yu changed the title Fixes #38150 - Switch log level of 2 harmless messages to info level Fixes #38150 - Switch log level of 2 harmless messages to debug level Jan 27, 2025
@hao-yu
Copy link
Contributor Author

hao-yu commented Jan 27, 2025

Sure! sounds good to me

Copy link
Member

@ianballou ianballou left a comment

Choose a reason for hiding this comment

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

Looks great, thanks!

@ianballou ianballou merged commit 882b257 into Katello:master Jan 27, 2025
19 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