X Tutup
Skip to content

add --signal option to stop and restart#3614

Merged
thaJeztah merged 1 commit intodocker:masterfrom
thaJeztah:signal_options
May 17, 2022
Merged

add --signal option to stop and restart#3614
thaJeztah merged 1 commit intodocker:masterfrom
thaJeztah:signal_options

Conversation

@thaJeztah
Copy link
Member

@thaJeztah thaJeztah commented May 17, 2022

Wording and documentation still need to be updated, but will do
so in a follow-up.

Also removing the default "10 seconds" from the timeout flags, as
this default is not actually used, and may not match the actual
default (which is defined on the daemon side).

- Description for the changelog

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

Wording and documentation still need to be updated, but will do
so in a follow-up.

Also removing the default "10 seconds" from the timeout flags, as
this default is not actually used, and may not match the actual
default (which is defined on the daemon side).

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah
Copy link
Member Author

I have a branch with "WIP" documentation changes, and changes to the completion scripts, but will push that as a follow-up pull request (some more work needed).

@codecov-commenter
Copy link

Codecov Report

Merging #3614 (86c30e6) into master (53f8ed4) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #3614      +/-   ##
==========================================
- Coverage   58.96%   58.96%   -0.01%     
==========================================
  Files         288      288              
  Lines       24421    24425       +4     
==========================================
  Hits        14401    14401              
- Misses       9148     9152       +4     
  Partials      872      872              

@thaJeztah thaJeztah merged commit b655203 into docker:master May 17, 2022
@thaJeztah thaJeztah deleted the signal_options branch May 17, 2022 12:31
matt9ucci added a commit to matt9ucci/DockerCompletion that referenced this pull request Feb 8, 2023
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.

Proposal: docker stop should allow you to specify which initial signal to use, instead of always using SIGTERM

4 participants

X Tutup