Skip to content

fix: send logging to stderr#31191

Merged
mattfarina merged 1 commit into
helm:mainfrom
TerryHowe:stderr-logging
Aug 26, 2025
Merged

fix: send logging to stderr#31191
mattfarina merged 1 commit into
helm:mainfrom
TerryHowe:stderr-logging

Conversation

@TerryHowe

Copy link
Copy Markdown
Contributor

What this PR does / why we need it:

if people are depending on the stdout of commands, the logging could disturb that.

Closes: #31188

Special notes for your reviewer:

If applicable:

  • this PR contains user facing changes (the docs needed label should be applied if so)
  • this PR contains unit tests
  • this PR has been tested for backwards compatibility

@pull-request-size pull-request-size Bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Aug 26, 2025

@mattfarina mattfarina left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm

@mattfarina
mattfarina merged commit 4ae80c0 into helm:main Aug 26, 2025
5 checks passed
@scottrigby scottrigby added the bug Categorizes issue or PR as related to a bug. label Nov 5, 2025
@scottrigby scottrigby added this to the 4.0.0 milestone Nov 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Categorizes issue or PR as related to a bug. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

spurious WARNING due to new plugin check

3 participants