Skip to content

Changed default stopsignal from SIGTERM to SIGINT #763

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Sep 23, 2020
Merged

Changed default stopsignal from SIGTERM to SIGINT #763

merged 1 commit into from
Sep 23, 2020

Conversation

GameScripting
Copy link
Contributor

Changed the stopsignal to SIGINT, to ensure the database data is always synced to disk, even if clients do not cooperate.
For Details see #714

@tianon
Copy link
Member

tianon commented Sep 18, 2020

I've updated this per the discussion in #714, especially to add it to all versions/variants, and to add a long-form comment explaining the rationale and pointing to the upstream documentation that's informed our choice (and can further inform users whose use case/constraints warrant making a different choice).

(We should probably add something similar to the image description once this lands.)

Verified

This commit was signed with the committer’s verified signature.
pietroalbini Pietro Albini
@tianon tianon requested a review from yosifkit September 23, 2020 00:12
@yosifkit yosifkit merged commit 2d0d9a3 into docker-library:master Sep 23, 2020
docker-library-bot added a commit to docker-library-bot/official-images that referenced this pull request Sep 24, 2020

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Changes:

- docker-library/postgres@2d0d9a3: Merge pull request docker-library/postgres#763 from GameScripting/patch-1
- docker-library/postgres@bfc5d81: Change default STOPSIGNAL from SIGTERM to SIGINT
docker-library-bot added a commit to docker-library-bot/official-images that referenced this pull request Sep 24, 2020
Changes:

- docker-library/postgres@8d28111: Update to 13.0-1.pgdg100+1
- docker-library/postgres@a7ec9d0: Update to 13.0
- docker-library/postgres@2d0d9a3: Merge pull request docker-library/postgres#763 from GameScripting/patch-1
- docker-library/postgres@bfc5d81: Change default STOPSIGNAL from SIGTERM to SIGINT
docker-library-bot added a commit to docker-library-bot/official-images that referenced this pull request Sep 24, 2020
Changes:

- docker-library/postgres@9abfeee: Update "latest" to 13 (now GA)
- docker-library/postgres@8d28111: Update to 13.0-1.pgdg100+1
- docker-library/postgres@a7ec9d0: Update to 13.0
- docker-library/postgres@2d0d9a3: Merge pull request docker-library/postgres#763 from GameScripting/patch-1
- docker-library/postgres@bfc5d81: Change default STOPSIGNAL from SIGTERM to SIGINT
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.

None yet

3 participants