GNU bug report logs - #70028
[PATCH] Switch order of the default substitute servers.

Previous Next

Package: guix-patches;

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

Date: Wed, 27 Mar 2024 13:44: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 70028 in the body.
You can then email your comments to 70028 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 <at> cbaines.net, dev <at> jpoiret.xyz, ludo <at> gnu.org, othacehe <at> gnu.org, rekado <at> elephly.net, zimon.toutoune <at> gmail.com, me <at> tobias.gr, guix-patches <at> gnu.org:
bug#70028; Package guix-patches. (Wed, 27 Mar 2024 13:44: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 <at> cbaines.net, dev <at> jpoiret.xyz, ludo <at> gnu.org, othacehe <at> gnu.org, rekado <at> elephly.net, zimon.toutoune <at> gmail.com, me <at> tobias.gr, guix-patches <at> gnu.org. (Wed, 27 Mar 2024 13:44: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] Switch order of the default substitute servers.
Date: Wed, 27 Mar 2024 13:43:43 +0000
The aim here is to improve the user experience.  There's anecdotal evidence
that the network performance for bordeaux is better compared to ci at least
for some users, and I don't know of any issues with rate limiting or access
restriction for bordeaux compared to ci.  It also has IPv6 support.

Additionally, bordeaux generally had more substitutes than ci, particularly
for aarch64-linux and armhf-linux.  This change will offer a very slight
speedup for those substitutes that only bordeaux has.

Bordeaux has been a default substitute server for nearly 3 years now and I
think this change is overdue.  I'm also hopeful that we'll be able to build on
the testing regarding mirrors for bordeaux, and that'll allow potentially
improving the hosting setup (through providing more redundancy) and further
improving substitute fetching for users who currently have issues with
substitute access.

* config-daemon.ac: Switch substitute urls order.
* doc/guix.texi: Ditto.
* etc/guix-install.sh: Ditto.
* gnu/installer/newt/network.scm (wait-service-online): Ditto.
* guix/store.scm (%default-substitute-urls): Ditto.

Change-Id: I4f6d93ae1fc8b03d80b47b18b5749a51f1fde17b
---
 config-daemon.ac               | 2 +-
 doc/guix.texi                  | 4 ++--
 etc/guix-install.sh            | 2 +-
 gnu/installer/newt/network.scm | 4 ++--
 guix/scripts/substitute.scm    | 4 ++--
 guix/store.scm                 | 4 ++--
 6 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/config-daemon.ac b/config-daemon.ac
index 86306effe1..9188f93bda 100644
--- a/config-daemon.ac
+++ b/config-daemon.ac
@@ -106,7 +106,7 @@ if test "x$guix_build_daemon" = "xyes"; then
 
   dnl Determine the appropriate default list of substitute URLs (GnuTLS
   dnl is required so we can default to 'https'.)
-  guix_substitute_urls="https://ci.guix.gnu.org https://bordeaux.guix.gnu.org"
+  guix_substitute_urls="https://bordeaux.guix.gnu.org https://ci.guix.gnu.org"
 
   AC_MSG_CHECKING([for default substitute URLs])
   AC_MSG_RESULT([$guix_substitute_urls])
diff --git a/doc/guix.texi b/doc/guix.texi
index ddd98a5fd4..ee3d5f5c3c 100644
--- a/doc/guix.texi
+++ b/doc/guix.texi
@@ -17,8 +17,8 @@
 @set BASE-URL https://ftp.gnu.org/gnu/guix
 
 @c The official substitute server used by default.
-@set SUBSTITUTE-SERVER-1 ci.guix.gnu.org
-@set SUBSTITUTE-SERVER-2 bordeaux.guix.gnu.org
+@set SUBSTITUTE-SERVER-1 bordeaux.guix.gnu.org
+@set SUBSTITUTE-SERVER-2 ci.guix.gnu.org
 @set SUBSTITUTE-URLS https://@value{SUBSTITUTE-SERVER-1} https://@value{SUBSTITUTE-SERVER-2}
 
 @copying
diff --git a/etc/guix-install.sh b/etc/guix-install.sh
index 982fb0a266..dde35e6d39 100755
--- a/etc/guix-install.sh
+++ b/etc/guix-install.sh
@@ -503,8 +503,8 @@ sys_enable_guix_daemon()
 sys_authorize_build_farms()
 { # authorize the public key(s) of the build farm(s)
     local hosts=(
-	ci.guix.gnu.org
 	bordeaux.guix.gnu.org
+	ci.guix.gnu.org
     )
 
     if prompt_yes_no "Permit downloading pre-built package binaries from the \
diff --git a/gnu/installer/newt/network.scm b/gnu/installer/newt/network.scm
index ba26fc7c76..b22cc71305 100644
--- a/gnu/installer/newt/network.scm
+++ b/gnu/installer/newt/network.scm
@@ -127,8 +127,8 @@ (define (wait-service-online)
           (lambda _ #f))
         (alarm 3))
       (lambda ()
-        (or (url-alive? "https://ci.guix.gnu.org")
-            (url-alive? "https://bordeaux.guix.gnu.org")))
+        (or (url-alive? "https://bordeaux.guix.gnu.org")
+            (url-alive? "https://ci.guix.gnu.org")))
       (lambda ()
         (alarm 0))))
 
diff --git a/guix/scripts/substitute.scm b/guix/scripts/substitute.scm
index 37cd08e289..90ce1be76c 100755
--- a/guix/scripts/substitute.scm
+++ b/guix/scripts/substitute.scm
@@ -749,8 +749,8 @@ (define %default-substitute-urls
     (#f
      ;; This can only happen when this script is not invoked by the
      ;; daemon.
-     '("http://ci.guix.gnu.org"
-       "http://bordeaux.guix.gnu.org"))))
+     '("http://bordeaux.guix.gnu.org"
+       "http://ci.guix.gnu.org"))))
 
 ;; In order to prevent using large number of discovered local substitute
 ;; servers, limit the local substitute urls list size.
diff --git a/guix/store.scm b/guix/store.scm
index 97c4f32a5b..1229198b09 100644
--- a/guix/store.scm
+++ b/guix/store.scm
@@ -767,8 +767,8 @@ (define %default-substitute-urls
   (map (if (false-if-exception (resolve-interface '(gnutls)))
            (cut string-append "https://" <>)
            (cut string-append "http://" <>))
-       '("ci.guix.gnu.org"
-         "bordeaux.guix.gnu.org")))
+       '("bordeaux.guix.gnu.org"
+         "ci.guix.gnu.org")))
 
 (define (current-user-name)
   "Return the name of the calling user."

base-commit: e9a547d2c57e22bea0b6da6cc67d423ff9d47b07
-- 
2.41.0





Information forwarded to guix-patches <at> gnu.org:
bug#70028; Package guix-patches. (Wed, 27 Mar 2024 14:25:03 GMT) Full text and rfc822 format available.

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

From: Ricardo Wurmus <rekado <at> elephly.net>
To: Christopher Baines <mail <at> cbaines.net>
Cc: 70028 <at> debbugs.gnu.org, debbugs-submit <at> debbugs.gnu.org,
 Simon Tournier <zimon.toutoune <at> gmail.com>, Mathieu Othacehe <othacehe <at> gnu.org>,
 Ludovic Courtès <ludo <at> gnu.org>,
 Tobias Geerinckx-Rice <me <at> tobias.gr>, Josselin Poiret <dev <at> jpoiret.xyz>,
 Christopher Baines <guix <at> cbaines.net>
Subject: Re: [bug#70028] [PATCH] Switch order of the default substitute
 servers.
Date: Wed, 27 Mar 2024 15:24:08 +0100
Christopher Baines <mail <at> cbaines.net> writes:

> The aim here is to improve the user experience.  There's anecdotal evidence
> that the network performance for bordeaux is better compared to ci at least
> for some users, and I don't know of any issues with rate limiting or access
> restriction for bordeaux compared to ci.  It also has IPv6 support.

LGTM

-- 
Ricardo




Information forwarded to guix-patches <at> gnu.org:
bug#70028; Package guix-patches. (Tue, 02 Apr 2024 15:35:02 GMT) Full text and rfc822 format available.

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

From: Ludovic Courtès <ludo <at> gnu.org>
To: Ricardo Wurmus <rekado <at> elephly.net>
Cc: 70028 <at> debbugs.gnu.org, Tobias Geerinckx-Rice <me <at> tobias.gr>,
 debbugs-submit <at> debbugs.gnu.org, Simon Tournier <zimon.toutoune <at> gmail.com>,
 Mathieu Othacehe <othacehe <at> gnu.org>, Christopher Baines <mail <at> cbaines.net>,
 Josselin Poiret <dev <at> jpoiret.xyz>, Christopher Baines <guix <at> cbaines.net>
Subject: Re: [bug#70028] [PATCH] Switch order of the default substitute
 servers.
Date: Tue, 02 Apr 2024 17:34:40 +0200
Hi,

Ricardo Wurmus <rekado <at> elephly.net> skribis:

> Christopher Baines <mail <at> cbaines.net> writes:
>
>> The aim here is to improve the user experience.  There's anecdotal evidence
>> that the network performance for bordeaux is better compared to ci at least
>> for some users, and I don't know of any issues with rate limiting or access
>> restriction for bordeaux compared to ci.  It also has IPv6 support.
>
> LGTM

No objections from me.

Note that it will take months to propagate to users, particularly users
on foreign distros who rarely update the daemon IME.

Thanks,
Ludo’.




Reply sent to Christopher Baines <mail <at> cbaines.net>:
You have taken responsibility. (Wed, 03 Apr 2024 17:15:02 GMT) Full text and rfc822 format available.

Notification sent to Christopher Baines <mail <at> cbaines.net>:
bug acknowledged by developer. (Wed, 03 Apr 2024 17:15:03 GMT) Full text and rfc822 format available.

Message #16 received at 70028-done <at> debbugs.gnu.org (full text, mbox):

From: Christopher Baines <mail <at> cbaines.net>
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: Ricardo Wurmus <rekado <at> elephly.net>, 70028-done <at> debbugs.gnu.org
Subject: Re: [bug#70028] [PATCH] Switch order of the default substitute
 servers.
Date: Wed, 03 Apr 2024 18:14:09 +0100
[Message part 1 (text/plain, inline)]
Ludovic Courtès <ludo <at> gnu.org> writes:

> Hi,
>
> Ricardo Wurmus <rekado <at> elephly.net> skribis:
>
>> Christopher Baines <mail <at> cbaines.net> writes:
>>
>>> The aim here is to improve the user experience.  There's anecdotal evidence
>>> that the network performance for bordeaux is better compared to ci at least
>>> for some users, and I don't know of any issues with rate limiting or access
>>> restriction for bordeaux compared to ci.  It also has IPv6 support.
>>
>> LGTM
>
> No objections from me.
>
> Note that it will take months to propagate to users, particularly users
> on foreign distros who rarely update the daemon IME.

Thanks both, I've pushed this to master as
ac19e038b457d0585812091200005befd9b29259.

Chris
[signature.asc (application/pgp-signature, inline)]

bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Thu, 02 May 2024 11:24:07 GMT) Full text and rfc822 format available.

This bug report was last modified 2 days ago.

Previous Next


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