Skip to content

Use debug instead of warning#68668

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

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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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

@twangboy twangboy linked an issue Feb 5, 2026 that may be closed by this pull request
@twangboy
Copy link
Copy Markdown
Contributor Author

Closing in favor of #68697

@twangboy twangboy closed this Feb 11, 2026
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