[bug#49364] [PATCH 0/2] Use log file for tor and ntp services

[bug#49364] [PATCH 0/2] Use log file for tor and ntp services

[Brice Waegeneire]

Brice Waegeneire (2):
  services: tor: Log to file.
  services: ntp: Log to file.

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

-- 
2.31.1

[bug#49364] [PATCH 1/2] services: tor: Log to file.

[Brice Waegeneire]

* gnu/services/networking.scm (tor-shepherd-service)[start]: Specify log
file.
---
 gnu/services/networking.scm | 1 +
 1 file changed, 1 insertion(+)

diff --git a/gnu/services/networking.scm b/gnu/services/networking.scm
index 1ae58041d3..0be4a21e7a 100644
--- a/gnu/services/networking.scm
+++ b/gnu/services/networking.scm
@@ -960,6 +960,7 @@ HiddenServicePort ~a ~a~%"
                 (start #~(make-forkexec-constructor/container
                           (list #$(file-append tor "/bin/tor") "-f" #$torrc)
 
+                          #:log-file "/var/log/tor.log"
                           #:mappings (list (file-system-mapping
                                             (source "/var/lib/tor")
                                             (target source)
-- 
2.31.1

[bug#49364] [PATCH 2/2] services: ntp: Log to file.

[Brice Waegeneire]

* gnu/services/networking.scm (ntp-shepherd-service)[start]: Specify log
file.
---
 gnu/services/networking.scm | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/gnu/services/networking.scm b/gnu/services/networking.scm
index 0be4a21e7a..eeb1487116 100644
--- a/gnu/services/networking.scm
+++ b/gnu/services/networking.scm
@@ -492,7 +492,8 @@ restrict source notrap nomodify noquery\n"))
                                 "-c" #$ntpd.conf "-u" "ntpd"
                                 #$@(if allow-large-adjustment?
                                        '("-g")
-                                       '()))))
+                                       '()))
+                          #:log-file "/var/log/ntpd.log"))
                 (stop #~(make-kill-destructor)))))))))
 
 (define %ntp-accounts
-- 
2.31.1

[bug#49364] [PATCH 0/2] Use log file for tor and ntp services

[Mathieu Othacehe]

Hey,

> Brice Waegeneire (2):
>   services: tor: Log to file.
>   services: ntp: Log to file.

It could be nice to add those log files to the ntp-configuration and the
tor-configuration records as users may want to have control on logging.

In the meantime, I guess you can proceed with this first implementation.

Thanks,

Mathieu

bug#49364: [PATCH 0/2] Use log file for tor and ntp services

[Brice Waegeneire]

Mathieu Othacehe <othacehe@gnu.org> writes:

>> Brice Waegeneire (2):
>>   services: tor: Log to file.
>>   services: ntp: Log to file.
>
> It could be nice to add those log files to the ntp-configuration and the
> tor-configuration records as users may want to have control on logging.

ATM I don't have use for specifying the log file path, I can review such
patch when someone write it.  I wrote this one mainly to avoid thoses two
services to write to tty1.

> In the meantime, I guess you can proceed with this first implementation.

Pushed asa5fa05dfc7e5bbe0191938b7824ec264aa842306 and
8bb45504dd9eb441d447e3f1dca174f0231c67d8.