Skip to content

flake: TestHealthSettings/UnDismissSection #194

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
johnstcn opened this issue Nov 4, 2024 · 2 comments
Open

flake: TestHealthSettings/UnDismissSection #194

johnstcn opened this issue Nov 4, 2024 · 2 comments
Labels

Comments

@johnstcn
Copy link
Member

johnstcn commented Nov 4, 2024

Seen here: https://github.com/coder/coder/actions/runs/11669113262/job/32490361705?pr=15364

    debug_test.go:328: 
        	Error Trace:	/Users/runner/work/coder/coder/coderd/debug_test.go:328
        	Error:      	Received unexpected error:
        	            	Get "http://localhost:53978/api/v2/debug/health": context deadline exceeded
        	Test:       	TestHealthSettings/UnDismissSection
@Emyrk
Copy link
Member

Emyrk commented Nov 4, 2024

First user http request: 17:09:41.951
Early test requests:

    t.go:99: 2024-11-04 17:09:41.953 [debu]  coderd: PUT  host=localhost:53978  path=/api/v2/debug/health/settings  proto=HTTP/1.1  remote_addr=127.0.0.1  start="2024-11-04T17:09:41.952115Z"  took=1.515916ms  status_code=200  latency_ms=1  request_id=865f68ab-dd4c-4dec-b5dc-af44f4f2acbb
    t.go:99: 2024-11-04 17:09:41.955 [debu]  coderd: PUT  host=localhost:53978  path=/api/v2/debug/health/settings  proto=HTTP/1.1  remote_addr=127.0.0.1  start="2024-11-04T17:09:41.954785Z"  took=664.667µs  status_code=200  latency_ms=0  request_id=2091fe81-ce81-4566-8ba9-8619b5d971bb
    t.go:99: 2024-11-04 17:09:41.956 [debu]  coderd: GET  host=localhost:53978  path=/api/v2/debug/health/settings  proto=HTTP/1.1  remote_addr=127.0.0.1  start="2024-11-04T17:09:41.956254Z"  took=140.666µs  status_code=200  latency_ms=0  request_id=80a40df2-c44d-45ad-836a-b4da2afa3bd9

Logs just jump 10s:

    t.go:99: 2024-11-04 17:09:42.008 [debu]  coderd: GET  host=127.0.0.1:53978  path=/derp/latency-check  proto=HTTP/1.1  remote_addr=127.0.0.1  start="2024-11-04T17:09:42.008329Z"  took=45.666µs  status_code=200  latency_ms=0  request_id=f8d8fec9-adf7-4dc0-8bad-381fa18451cb
    debug_test.go:328: 
        	Error Trace:	/Users/runner/work/coder/coder/coderd/debug_test.go:328
        	Error:      	Received unexpected error:
        	            	Get "http://localhost:53978/api/v2/debug/health": context deadline exceeded
        	Test:       	TestHealthSettings/UnDismissSection
    t.go:99: 2024-11-04 17:09:51.916 [debu]  coderd.servertailnet: polling DERPMap exited

I don't see any reason why the test would block for 10s 🤔

@johnstcn
Copy link
Member Author

johnstcn commented Nov 4, 2024

I wonder if it was the DERP health report that took longer than 10s to complete?
I think half the issue here is that the hc is run on request + cached instead of being run in the background.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants