Actions
Bug #21927
closedlib/controller/handler_test.go TestDiscoveryDocCache fails intermittently
Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Tests
Target version:
Story points:
-
Release:
Release relationship:
Auto
Description
time="2024-06-20T21:26:27Z" level=warning msg="error refreshing cache" error="HTTP status 400" path=/discovery/v1/apis/arvados/v1/rest ---------------------------------------------------------------------- FAIL: handler_test.go:120: HandlerSuite.TestDiscoveryDocCache handler_test.go:278: // Only one attempt to re-fetch (holdReqs ensured the first // update took long enough for the last incoming request to // arrive) c.Check(countRailsReqs(), check.Equals, reqsBefore+1) ... obtained int = 10 ... expected int = 11
Note that we previously had #21276 to do some work on this. As far as I'm aware this is the first time we've seen a problem since that branch was merged six months ago.
Files
Related issues
Updated by Brett Smith 5 months ago
- Related to Bug #21276: lib/controller/handler_test.go TestDiscoveryDocCache is racey added
Updated by Brett Smith 4 months ago
- Has duplicate Bug #21991: Unreliable test HandlerSuite.TestDiscoveryDocCache in lib/controller added
Updated by Brett Smith 4 months ago
- Target version set to Development 2024-07-24 sprint
- Assigned To set to Tom Clegg
Updated by Peter Amstutz 4 months ago
- Target version changed from Development 2024-07-24 sprint to Development 2024-08-07 sprint
Updated by Tom Clegg 4 months ago
- Status changed from New to In Progress
I was able to reproduce this by inserting a time.Sleep()
at the top of (*cacheEnt)refresh()
. Fix is explained in the commit message & comments.
21927-discovery-doc-test @ 8a633aa6872257548c4b63156a0031e3e74244ec -- developer-run-tests: #4364
Updated by Brett Smith 4 months ago
Tom Clegg wrote in #note-5:
21927-discovery-doc-test @ 8a633aa6872257548c4b63156a0031e3e74244ec -- developer-run-tests: #4364
LGTM, thanks.
Updated by Tom Clegg 4 months ago
- Status changed from In Progress to Resolved
Applied in changeset arvados|23917df17e468d3a1f80c67645c9ba7a32ca6761.
Actions