[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#48468: substitute server connection timeout
From: |
Mathieu Othacehe |
Subject: |
bug#48468: substitute server connection timeout |
Date: |
Tue, 27 Dec 2022 10:52:51 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) |
Hey Ludo,
> That’s still below the 100 MiB cache bypass threshold of the main ‘guix
> publish’ instance though.
Right. Just to be on the safe side here, what about applying this patch
to have log lines when we are replying 404 due to baking?
Thanks,
Mathieu
>From 725d5ba21a0fc0108b60c37bbc8d947fab6ac938 Mon Sep 17 00:00:00 2001
From: Mathieu Othacehe <othacehe@gnu.org>
Date: Tue, 27 Dec 2022 10:49:04 +0100
Subject: [PATCH 1/1] scripts: publish: Add a log when replying 404 due to
baking.
* guix/scripts/publish.scm (render-narinfo/cached): Add it.
---
guix/scripts/publish.scm | 10 +++++++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/guix/scripts/publish.scm b/guix/scripts/publish.scm
index 3bf3bd9c7c..a2048c98fd 100644
--- a/guix/scripts/publish.scm
+++ b/guix/scripts/publish.scm
@@ -586,9 +586,13 @@ (define (delete-entry narinfo)
#:ttl 300 ;temporary
#:nar-path nar-path
#:compressions compressions)
- (not-found request
- #:phrase "We're baking it"
- #:ttl 300))) ;should be available within 5m
+ (begin
+ (format #t (G_ "~a ~a: 404 (baking)~%")
+ (request-method request)
+ (uri-path (request-uri request)))
+ (not-found request
+ #:phrase "We're baking it"
+ #:ttl 300)))) ;should be available within 5m
(else
(not-found request #:phrase "" #:ttl negative-ttl)))))
--
2.38.1