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

[nginx] Remove systemd override for nginx.service #2427

Merged
merged 1 commit into from
Oct 28, 2024

Conversation

Alphix
Copy link
Contributor

@Alphix Alphix commented Oct 2, 2023

Since Debian package version 1.22.0-2 (Bookworm, Mantic), the nginx.service unit has been updated to make "After=network-online.target" the default, making this override unnecessary.

@Alphix Alphix requested a review from drybjed as a code owner October 2, 2023 09:11
Since Debian package version 1.22.0-2 (Bookworm, Mantic), the nginx.service
unit has been updated to make "After=network-online.target" the default,
making this override unnecessary.
@Alphix Alphix force-pushed the nginx_remove_systemd_override branch from 78b726d to a05fe4e Compare October 15, 2023 12:16
Copy link
Member

@drybjed drybjed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks! I'll backport this to the stable releases.

@drybjed drybjed added change requests to change existing functionality priority: medium status: completed whatever was proposed is now done tag: nginx labels Oct 28, 2024
@drybjed drybjed added this to the DebOps v3.3.0 milestone Oct 28, 2024
@drybjed drybjed merged commit 76ed363 into debops:master Oct 28, 2024
14 of 15 checks passed
@Alphix Alphix deleted the nginx_remove_systemd_override branch November 5, 2024 19:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
change requests to change existing functionality priority: medium status: completed whatever was proposed is now done tag: nginx
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants