diff mbox series

[bug#63417] services: guix-build-coordinator: Set GC_RETRY_SIGNALS=0.

Message ID 77bc127265325bc2c593abee09d08b79dc578566.1683722725.git.mail@cbaines.net
State New
Headers
Series [bug#63417] services: guix-build-coordinator: Set GC_RETRY_SIGNALS=0. |

Commit Message

Christopher Baines May 10, 2023, 12:45 p.m. UTC 
This seems to work around the crashes/restarts that have recently been
affecting the coordinator (see #63368).

* gnu/services/guix.scm (guix-build-coordinator-shepherd-services): Set
GC_RETRY_SIGNALS=0.
---
 gnu/services/guix.scm | 5 +++++
 1 file changed, 5 insertions(+)


base-commit: 98c393164167e9a5a205f7dffa966992a71925ae
prerequisite-patch-id: 1a081b728996250cec703888ab2b5aad184e616d

Comments

Maxim Cournoyer Sept. 1, 2023, 2:21 p.m. UTC | #1 
Hi Christopher,

Christopher Baines <mail@cbaines.net> writes:

> This seems to work around the crashes/restarts that have recently been
> affecting the coordinator (see #63368).
>
> * gnu/services/guix.scm (guix-build-coordinator-shepherd-services): Set
> GC_RETRY_SIGNALS=0.
> ---
>  gnu/services/guix.scm | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/gnu/services/guix.scm b/gnu/services/guix.scm
> index 90310ab9d9..f8fee38b7a 100644
> --- a/gnu/services/guix.scm
> +++ b/gnu/services/guix.scm
> @@ -372,6 +372,11 @@ (define (guix-build-coordinator-shepherd-services config)
>                           "GUIX_LOCPATH=" #$glibc-utf8-locales "/lib/locale")
>                         "LC_ALL=en_US.utf8"
>                         "PATH=/run/current-system/profile/bin" ; for hooks
> +                       ;; TODO: This seems to help with avoid
> +                       ;; crashes/restarts, but ideally the Guix service
> +                       ;; wouldn't be setting environment variables to fiddle
> +                       ;; with the behaviour of Guile's GC.
> +                       "GC_RETRY_SIGNALS=0"
>                         #$@extra-environment-variables)
>                       #:log-file "/var/log/guix-build-coordinator/coordinator.log")
>                      args))))

Have you tried presenting the problem to more core Guile folks?
(guile-devel).  Perhaps someone could tip you in the right direction, or
perhaps it's a bug in Guile?
diff mbox series

Patch

diff --git a/gnu/services/guix.scm b/gnu/services/guix.scm
index 90310ab9d9..f8fee38b7a 100644
--- a/gnu/services/guix.scm
+++ b/gnu/services/guix.scm
@@ -372,6 +372,11 @@  (define (guix-build-coordinator-shepherd-services config)
                          "GUIX_LOCPATH=" #$glibc-utf8-locales "/lib/locale")
                        "LC_ALL=en_US.utf8"
                        "PATH=/run/current-system/profile/bin" ; for hooks
+                       ;; TODO: This seems to help with avoid
+                       ;; crashes/restarts, but ideally the Guix service
+                       ;; wouldn't be setting environment variables to fiddle
+                       ;; with the behaviour of Guile's GC.
+                       "GC_RETRY_SIGNALS=0"
                        #$@extra-environment-variables)
                      #:log-file "/var/log/guix-build-coordinator/coordinator.log")
                     args))))