mbox series

[bug#58590,0/2] Rework nginx-shepherd reload

Message ID 20221017205719.5704-1-eu@euandre.org
Headers show
Series Rework nginx-shepherd reload | expand

Message

EuAndreh Oct. 17, 2022, 8:57 p.m. UTC 
As discussed in [0], improve the documentation of the "reload" action
of nginx, to avoid letting users get into the pitfall of thinking that
a this changes the configuration file, or enables a configuration file
change to be deployed.

[0]: https://issues.guix.gnu.org/58405

EuAndreh (2):
  services: nginx: Use nginx-action over inline gexp.
  services: nginx: Improve reload action documentation.

 gnu/services/web.scm | 14 ++++++--------
 1 file changed, 6 insertions(+), 8 deletions(-)

Comments

Christopher Baines Oct. 18, 2022, 10:16 a.m. UTC | #1 
EuAndreh <eu@euandre.org> writes:

> As discussed in [0], improve the documentation of the "reload" action
> of nginx, to avoid letting users get into the pitfall of thinking that
> a this changes the configuration file, or enables a configuration file
> change to be deployed.
>
> [0]: https://issues.guix.gnu.org/58405
>
> EuAndreh (2):
>   services: nginx: Use nginx-action over inline gexp.
>   services: nginx: Improve reload action documentation.
>
>  gnu/services/web.scm | 14 ++++++--------
>  1 file changed, 6 insertions(+), 8 deletions(-)

Thanks, I've pushed these two commits to master now (as
117b0a65c832dcec592a64284910c01f595aa84a and
dbdc8f64e7e2d35abbfbfc40ac06cf103207d99e).

Thanks again,

Chris