[ARVADOS] updated: 2.1.0-1404-gfbbafc5ec
Git user
git at public.arvados.org
Thu Sep 23 21:02:31 UTC 2021
Summary of changes:
lib/controller/auth_test.go | 8 ++--
lib/controller/federation_test.go | 8 ++--
lib/controller/server_test.go | 8 ++--
lib/service/cmd.go | 5 +-
sdk/go/httpserver/logger.go | 41 +++++++++++++++--
sdk/go/httpserver/logger_test.go | 63 ++++++++++++++++++++++++--
services/api/config/arvados_config.rb | 1 +
services/api/config/initializers/db_timeout.rb | 9 ++++
services/keep-web/server.go | 10 +++-
9 files changed, 133 insertions(+), 20 deletions(-)
create mode 100644 services/api/config/initializers/db_timeout.rb
via fbbafc5ecb48485913cec69d9dd45ffd42c99cef (commit)
via 747de7f566c347eeff252b98c9aa09271e7fa24a (commit)
via fdd3dd304439a7912db1c6ef257cf512ec7b3b52 (commit)
via fc1c9a3d80a5fc71142ad11f4eda05c26e351569 (commit)
via 2b8fc576e242c0b8658eef9f1130143e009efc4d (commit)
via d90f02c0e969b649da61fdd272a9aab69c487de3 (commit)
from 190273b4931aff39d7aba26cf7e7e7e6be1242ff (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
commit fbbafc5ecb48485913cec69d9dd45ffd42c99cef
Merge: 190273b49 747de7f56
Author: Tom Clegg <tom at curii.com>
Date: Thu Sep 23 17:01:53 2021 -0400
Merge branch '13697-database-timeout'
fixes #13697
Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom at curii.com>
commit 747de7f566c347eeff252b98c9aa09271e7fa24a
Author: Tom Clegg <tom at curii.com>
Date: Thu Sep 23 17:01:09 2021 -0400
13967: Add guard comment re timing out large uploads.
Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom at curii.com>
diff --git a/services/keep-web/server.go b/services/keep-web/server.go
index b0375ff8d..586f6b805 100644
--- a/services/keep-web/server.go
+++ b/services/keep-web/server.go
@@ -25,6 +25,11 @@ func (srv *server) Start(logger *logrus.Logger) error {
h := &handler{Config: srv.Config}
reg := prometheus.NewRegistry()
h.Config.Cache.registry = reg
+ // Warning: when updating this to use Command() from
+ // lib/service, make sure to implement an exemption in
+ // httpserver.HandlerWithDeadline() so large file uploads are
+ // allowed to take longer than the usual API.RequestTimeout.
+ // See #13697.
mh := httpserver.Instrument(reg, logger, httpserver.AddRequestIDs(httpserver.LogRequests(h)))
h.MetricsAPI = mh.ServeAPI(h.Config.cluster.ManagementToken, http.NotFoundHandler())
srv.Handler = mh
-----------------------------------------------------------------------
hooks/post-receive
--
More information about the arvados-commits
mailing list