Skip to content

fix: forward SIGTERM to legacy CLI plugins#7065

Draft
maxpetrusenkoagent wants to merge 1 commit into
docker:masterfrom
maxpetrusenkoagent:hermes/oss-pr-2026-06-20-docker-cli-4402
Draft

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

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