Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Update s3s.py to allow clean docker exit #196

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update s3s.py to allow clean docker exit #196

wants to merge 1 commit into from

Commits on Oct 26, 2024

  1. Update s3s.py to allow clean docker exit 

    Process sigterm to allow the docker container to cleanly exit.
https://stackoverflow.com/questions/62853875/stopping-python-container-is-slow-sigterm-not-passed-to-python-process


"Docker runs your application, per default, in foreground, so, as PID 1, this said, the process with the PID 1 as a special meaning and specific protections in Linux.

This is highlighted in docker run documentation:

    Note

    A process running as PID 1 inside a container is treated specially by Linux: it ignores any signal with the default action. As a result, the process will not terminate on SIGINT or SIGTERM unless it is coded to do so."
    @NaruZosa
    NaruZosa authored Oct 26, 2024
    Configuration menu
    Copy the full SHA
    3e76827 View commit details
    Browse the repository at this point in the history