Page MenuHomeDevCentral
Differential D3626

Permit safe reload of nginx
ClosedPublic
Actions

Authored by DorianWinty on Mar 12 2025, 21:29.
Tags
None
Referenced Files
F36814662: D3626.diff
Sat, Jul 18, 03:49
F36806187: D3626.id9379.diff
Sat, Jul 18, 03:13
F36770196: D3626.id9378.diff
Sat, Jul 18, 00:14
F36770087: D3626.id9387.diff
Sat, Jul 18, 00:13
F36741724: D3626.id9387.diff
Fri, Jul 17, 21:51
F36726339: D3626.id9378.diff
Fri, Jul 17, 20:37
F36726334: D3626.id9387.diff
Fri, Jul 17, 20:37
F36726064: D3626.diff
Fri, Jul 17, 20:36
View All Files
Subscribers
None

Details

Reviewers
dereckson
Maniphest Tasks
T2098: acme.sh configure provisionning of the nginx check and reload
Commits
rOPSb73f67ee791e: Permit safe reload of nginx
Summary

Ref T2098

Test Plan

Deploy this diff and try to renew a certificate how have the script added

Diff Detail

Repository
rOPS Nasqueron Operations
Lint
Lint Passed
Unit
No Test Coverage
Branch
reloadCheck
Build Status
Buildable 5789
Build 6071: arc lint + arc unit

Event Timeline

DorianWinty requested review of this revision.Mar 12 2025, 21:29
DorianWinty created this revision.
Harbormaster completed remote builds in B5789: Diff 9378.Mar 12 2025, 21:29
dereckson requested changes to this revision.Mar 13 2025, 17:31

Looks good to me, source file to update.

roles/core/certificates/files/acmesh-nginxCheck.sh
8

New path?

8

?

This revision now requires changes to proceed.Mar 13 2025, 17:31
DorianWinty updated this revision to Diff 9379.Mar 26 2025, 18:02
DorianWinty marked 2 inline comments as done.

path update

Harbormaster completed remote builds in B5790: Diff 9379.Mar 26 2025, 18:02
dereckson accepted this revision.Mar 27 2025, 19:54
This revision is now accepted and ready to land.Mar 27 2025, 19:54
Closed by commit rOPSb73f67ee791e: Permit safe reload of nginx (authored by DorianWinty). · Explain WhyMar 27 2025, 20:32
This revision was automatically updated to reflect the committed changes.
DorianWinty added a commit: rOPSb73f67ee791e: Permit safe reload of nginx.

Revision Contents
Changeset List

PathSize
roles/
core/
certificates/
7 lines
files/
29 lines
1 line
sudo/
files/
16 lines
9 lines

Diff 9378

View Options

roles/core/certificates/acmesh.sls

Loading...
View Options

roles/core/certificates/files/acmesh-nginxCheck.sh

Loading...
View Options

roles/core/certificates/init.sls

Loading...
View Options

roles/core/sudo/files/acme

Loading...
View Options

roles/core/sudo/init.sls

Loading...
Nasqueron DevCentral · If it had been much bigger the moon would have had a core of ice. · Powered by Phabricator