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.

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

[bitnami/nginx] shutting down gracefully #75942

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

Conversation

tlserver
Copy link

@tlserver tlserver commented Jan 2, 2025

Description of the Change

This pull request adds STOPSIGNAL SIGQUIT to the Dockerfile for Nginx. This change allows Nginx to handle shutdown signals more gracefully.

Benefits

By implementing this change, when docker stop is called, Nginx will be able to shut down gracefully, improving service reliability and user experience during container shutdowns.

Possible Drawbacks

If immediate termination of Nginx is required, users must specify --stop-signal SIGTERM when running or creating containers. This retains flexibility for users who prefer the default behavior.

Applicable Issues

Additional Information

For more details on signal handling in Nginx, please refer to the official documentation: Nginx Runtime Control.

@github-actions github-actions bot added nginx triage Triage is needed labels Jan 2, 2025
@github-actions github-actions bot requested a review from javsalgar January 2, 2025 02:44
@tlserver tlserver force-pushed the 65691-gracefully-shutdown branch from 0724bb8 to 4951fc1 Compare January 2, 2025 08:29
@carrodher carrodher added verify Execute verification workflow for these changes in-progress labels Jan 2, 2025
@github-actions github-actions bot removed the triage Triage is needed label Jan 2, 2025
@github-actions github-actions bot removed the request for review from javsalgar January 2, 2025 08:34
@github-actions github-actions bot requested a review from migruiz4 January 2, 2025 08:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in-progress nginx verify Execute verification workflow for these changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[bitnami/nginx] Nginx not shutting down gracefully by-default
4 participants