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

Previous Next

Package: guix-patches;

Reported by: Christopher Baines <mail <at> cbaines.net>

Date: Wed, 10 May 2023 12:46:02 UTC

Severity: normal

Tags: patch

Done: Christopher Baines <mail <at> cbaines.net>

Bug is archived. No further changes may be made.

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 63417 in the body.
You can then email your comments to 63417 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to guix-patches <at> gnu.org:
bug#63417; Package guix-patches. (Wed, 10 May 2023 12:46:02 GMT) Full text and rfc822 format available.

Acknowledgement sent to Christopher Baines <mail <at> cbaines.net>:
New bug report received and forwarded. Copy sent to guix-patches <at> gnu.org. (Wed, 10 May 2023 12:46:02 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Christopher Baines <mail <at> cbaines.net>
To: guix-patches <at> gnu.org
Subject: [PATCH] services: guix-build-coordinator: Set GC_RETRY_SIGNALS=0.
Date: Wed, 10 May 2023 13:45:25 +0100
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))))

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





Information forwarded to guix-patches <at> gnu.org:
bug#63417; Package guix-patches. (Fri, 01 Sep 2023 14:23:01 GMT) Full text and rfc822 format available.

Message #8 received at 63417 <at> debbugs.gnu.org (full text, mbox):

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Christopher Baines <mail <at> cbaines.net>
Cc: 63417 <at> debbugs.gnu.org
Subject: Re: bug#63417: [PATCH] services: guix-build-coordinator: Set
 GC_RETRY_SIGNALS=0.
Date: Fri, 01 Sep 2023 10:21:56 -0400
Hi Christopher,

Christopher Baines <mail <at> 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?

-- 
Thanks,
Maxim




Reply sent to Christopher Baines <mail <at> cbaines.net>:
You have taken responsibility. (Tue, 05 Sep 2023 11:21:01 GMT) Full text and rfc822 format available.

Notification sent to Christopher Baines <mail <at> cbaines.net>:
bug acknowledged by developer. (Tue, 05 Sep 2023 11:21:02 GMT) Full text and rfc822 format available.

Message #13 received at 63417-close <at> debbugs.gnu.org (full text, mbox):

From: Christopher Baines <mail <at> cbaines.net>
To: 63417-close <at> debbugs.gnu.org
Subject: Re: [bug#63417] [PATCH] services: guix-build-coordinator: Set
 GC_RETRY_SIGNALS=0.
Date: Tue, 05 Sep 2023 12:20:44 +0100



bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Tue, 03 Oct 2023 11:24:06 GMT) Full text and rfc822 format available.

This bug report was last modified 203 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.