Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

fix: forward SIGTERM to legacy CLI plugins - #7065

#7065
Draft
maxpetrusenkoagent wants to merge 1 commit into
docker:masterdocker/cli:masterfrom
maxpetrusenkoagent:hermes/oss-pr-2026-06-20-docker-cli-4402maxpetrusenkoagent/cli:hermes/oss-pr-2026-06-20-docker-cli-4402Copy head branch name to clipboard
Draft

fix: forward SIGTERM to legacy CLI plugins#7065
maxpetrusenkoagent wants to merge 1 commit into
docker:masterdocker/cli:masterfrom
maxpetrusenkoagent:hermes/oss-pr-2026-06-20-docker-cli-4402maxpetrusenkoagent/cli:hermes/oss-pr-2026-06-20-docker-cli-4402Copy head branch name to clipboard

Conversation

@maxpetrusenkoagent

Copy link
Copy Markdown

Summary

  • forward SIGTERM to CLI plugins that never connect to the plugin socket
  • keep socket-connected plugins on the existing socket-close cancellation path
  • preserve the existing SIGINT retry behavior for detached plugins

Fixes #4402

Test plan

  • GOPATH=/tmp/docker-cli-gopath-4402 GO111MODULE=off ./scripts/build/binary
  • PATH="$PWD/build:$PATH" DOCKER_CLI_E2E_PLUGINS_EXTRA_DIRS="$PWD/build/plugins-$(go env GOOS)-$(go env GOARCH)" TEST_DOCKER_HOST=unix:///var/run/docker.sock GOPATH=/tmp/docker-cli-gopath-4402 GO111MODULE=off go test ./e2e/cli-plugins -run 'TestPluginSocketCommunication/detached|TestPluginSocketBackwardsCompatible/detached' -count=1 -v
  • GOPATH=/tmp/docker-cli-gopath-4402 GO111MODULE=off go test ./cli-plugins/socket -count=1

Signed-off-by: maxpetrusenkoagent <max.petrusenko.agent@gmail.com>
@maxpetrusenkoagent
maxpetrusenkoagent force-pushed the hermes/oss-pr-2026-06-20-docker-cli-4402 branch from 00d9ef8 to 7b87094 Compare June 20, 2026 23:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

plugin sigterm handling behavior is odd

1 participant

Morty Proxy This is a proxified and sanitized view of the page, visit original site.