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

[3007.x] Change log level of publisher binding from error to debug #66268

Merged
merged 1 commit into from
Apr 10, 2024

Conversation

clayoster
Copy link
Contributor

What does this PR do?

Sets the log level of publisher binding from error to debug.

What issues does this PR fix or reference?

Fixes: #66179

Previous Behavior

Logs indicating publisher binding were being logged as errors

New Behavior

These logs will not be seen unless the master is running with a log_level of debug

Merge requirements satisfied?

[NOTICE] Bug fixes or features added to Salt require tests.

Commits signed with GPG?

Yes

@clayoster clayoster requested a review from a team as a code owner March 24, 2024 19:44
@clayoster clayoster requested review from dwoz and removed request for a team March 24, 2024 19:44
@salt-project-bot-prod-environment salt-project-bot-prod-environment bot changed the title Change log level of publisher binding from error to debug [3007.x] Change log level of publisher binding from error to debug Mar 24, 2024
@whytewolf
Copy link
Collaborator

need a test to make sure the proper log level is being used.

@tvb
Copy link

tvb commented Mar 28, 2024

Awaiting the merge of this PR to fix #66179

@dwoz dwoz merged commit 804cc62 into saltstack:3007.x Apr 10, 2024
502 checks passed
@clayoster clayoster deleted the master-publisher-error-logs branch April 10, 2024 22:57
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.

4 participants