Skip to content

cli/connhelper/commandcon: remove warn logs#6644

Merged
thaJeztah merged 1 commit intodocker:masterfrom
thaJeztah:connhelper_nowarn
Nov 10, 2025
Merged

cli/connhelper/commandcon: remove warn logs#6644
thaJeztah merged 1 commit intodocker:masterfrom
thaJeztah:connhelper_nowarn

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

These were originally added in 6f61cf0, but at the time, the error wasn't returned. Now that it is, we shouldn't log and return the error.

- What I did

- How I did it

- How to verify it

- Human readable description for the release notes

- A picture of a cute animal (not mandatory but encouraged)

These were originally added in 6f61cf0,
but at the time, the error wasn't returned. Now that it is, we shouldn't
log _and_ return the error.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah added this to the 29.0.0 milestone Nov 10, 2025
@thaJeztah thaJeztah added status/2-code-review kind/refactor PR's that refactor, or clean-up code labels Nov 10, 2025
@thaJeztah
Copy link
Copy Markdown
Member Author

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@thaJeztah thaJeztah merged commit 3d4129b into docker:master Nov 10, 2025
97 checks passed
@thaJeztah thaJeztah deleted the connhelper_nowarn branch November 10, 2025 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/refactor PR's that refactor, or clean-up code status/2-code-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants