Subject: gnu: Add #:log-file option when starting openvpn shepherd services
From: Cameron Chaparro <[email-masked]>
To: [email-masked]
Date: Thu, 24 Feb 2022 16:23:47 -0600
List-Id: <debbugs-submit.debbugs.gnu.org>

I accidentally submitted this to bug-guix, first, sorry.

---------- Forwarded message ---------
From: Cameron Chaparro <[email-masked]>
Date: Thu, Feb 24, 2022 at 4:11 PM
Subject: openvpn-shepherd-service is not writing log files
To: <[email-masked]>

I noticed that the `openvpn-shepherd-service' is defining, but not
actually using, the log file when starting the client/server. I think
this patch should address that issue.

Cameron Chaparro
From 372f3b8a2f2900bca5a55208a07940e6b70ddded Mon Sep 17 00:00:00 2001
Message-Id: <[email-masked]>
From: Cameron Chaparro <[email-masked]>
Date: Thu, 24 Feb 2022 15:49:15 -0600
Subject: [PATCH] gnu: Add #:log-file option when starting openvpn shepherd
 services

---
 gnu/services/vpn.scm | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/gnu/services/vpn.scm b/gnu/services/vpn.scm
index 3e370ba4be..b24e9cffb3 100644
--- a/gnu/services/vpn.scm
+++ b/gnu/services/vpn.scm
@@ -9,6 +9,7 @@
 ;;; Copyright © 2021 Raghav Gururajan <[email-masked]>
 ;;; Copyright © 2021 jgart <[email-masked]>
 ;;; Copyright © 2021 Nathan Dehnel <[email-masked]>
+;;; Copyright © 2022 Cameron V Chaparro <[email-masked]>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -494,7 +495,8 @@ (define (openvpn-shepherd-service role)
                        (list (string-append #$openvpn "/sbin/openvpn")
                              "--writepid" #$pid-file "--config" #$config-file
                              "--daemon")
-                       #:pid-file #$pid-file))
+                       #:pid-file #$pid-file
+                       #:log-file #$log-file))
              (stop #~(make-kill-destructor)))))))
 
 (define %openvpn-accounts

base-commit: 979b6a2c55fab729923d76ac9fee027d5d2d8f2b
-- 
2.34.0