Skip to content

cmd/docker: Add guidance for unavailable offload commands - #7369

Merged
thaJeztah merged 1 commit into
docker:masterfrom
nico1510:offload-plugin-guidance
Oct 9, 2026
Merged

thaJeztah merged 1 commit into
docker:masterfrom
nico1510:offload-plugin-guidance

Conversation

@nico1510

@nico1510 nico1510 commented Oct 9, 2026

Copy link
Copy Markdown
Member

Follow up to: #7358

Summary

Show installation guidance and a documentation link when docker offload commands cannot find a usable plugin, instead of the generic unknown-command error. This also covers docker offload --help, matching the guidance already provided for --cloud.

Validated with go test ./cmd/docker ./cli-plugins/manager and CLI checks for missing, invalid, and working plugins, including help behavior and exit codes.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
cmd/docker/docker.go 0.00% 1 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

@docker-agent docker-agent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Assessment: 🟢 APPROVE

Show installation guidance and a documentation link when the offload
plugin is missing or invalid. Handle this before command execution so
plugin-specific flags and --help also receive the guidance.

Signed-off-by: Nicolas Beck <nicolas.beck@docker.com>
@nico1510
nico1510 force-pushed the offload-plugin-guidance branch from c29717a to 3e686a8 Compare October 9, 2026 10:07
@nico1510
nico1510 marked this pull request as ready for review October 9, 2026 10:08

@docker-agent docker-agent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Assessment: 🟢 APPROVE

@thaJeztah thaJeztah left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@thaJeztah thaJeztah added this to the 29.9.1 milestone Oct 9, 2026
@thaJeztah
thaJeztah merged commit 02fbbf3 into docker:master Oct 9, 2026
107 of 109 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants