Skip to content

Use debug instead of warning#68668

Open
twangboy wants to merge 2 commits intosaltstack:3006.xfrom
twangboy:fix/VCOPS-65759/3006.x
Open

Use debug instead of warning#68668
twangboy wants to merge 2 commits intosaltstack:3006.xfrom
twangboy:fix/VCOPS-65759/3006.x

Conversation

@twangboy
Copy link
Contributor

@twangboy twangboy commented Jan 30, 2026

What does this PR do?

Fixes an issue where the minion log was being filled with warnings about maximum number of processes reached.

What issues does this PR fix or reference?

Fixes #68688 and Jira Ticket

Merge requirements satisfied?

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

Commits signed with GPG?

Yes

@twangboy twangboy requested a review from a team as a code owner January 30, 2026 04:33
@twangboy twangboy added the test:full Run the full test suite label Jan 30, 2026
@twangboy twangboy added this to the Sulphur v3006.20 milestone Jan 30, 2026
@twangboy twangboy self-assigned this Jan 30, 2026
Copy link
Contributor

@dwoz dwoz left a comment

Choose a reason for hiding this comment

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

We should open a github issue for this and add a changelog.

@frebib
Copy link
Contributor

frebib commented Feb 2, 2026

This sounds like something that should be a warning. Changing it to debug is simply just burying an error by turning a blind eye

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test:full Run the full test suite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Too many warnings about max num processes

4 participants