Flake: TestWorkspaceProxyWorkspaceApps_BlockDirect/* #112

Activity

1. ethanndickson

   added

   flake

   on Oct 17, 2024
2. ethanndickson

   assigned

   sreya

   on Oct 17, 2024
3. ethanndickson commented on Oct 17, 2024

   

   ethanndickson

   on Oct 17, 2024

   MemberAuthor

   More actions

   @sreya Assigning to you since it looks like the key rotation work changed workspace proxies a fair bit.
4. matifali

   added the

   Chore

   issue type on Apr 10, 2025
5. flake-investigator commented on Jan 8, 2026

   

   flake-investigatorBot

   on Jan 8, 2026

   More actions

   New occurrence on main CI (linux, test-go-pg-17)

   CI run: https://github.com/coder/coder/actions/runs/20830288896
   Job: test-go-pg-17
   Commit: 1e8c292855d95612417ccda801a7dd1956dfcd84 (Yevhenii Shcherbina)

   Failing test:

   • enterprise/wsproxy TestWorkspaceProxyWorkspaceApps_BlockDirect/WorkspaceAppsProxyPath/RedirectsWithQuery

   Key evidence:

   [erro] failed to deregister workspace proxy with Coder primary (it will be automatically deregistered shortly)
     error= make request:
            github.com/coder/coder/v2/enterprise/wsproxy/wsproxysdk.(*Client).DeregisterWorkspaceProxy
              /home/runner/work/coder/coder/enterprise/wsproxy/wsproxysdk/wsproxysdk.go:250
          - Post "http://127.0.0.1:37077/api/v2/workspaceproxies/me/deregister": net/http: HTTP/1.x transport connection broken: http: CloseIdleConnections called
    *** slogtest: log detected at level ERROR; TEST FAILURE ***
   

   Notes:

   • This looks like shutdown ordering / teardown noise: deregister fails because the HTTP transport is already being torn down, but we log it at ERROR and slogtest fails the test.
   • Potential fix: downgrade this log to warn/debug, or suppress it when shutting down / when error is caused by CloseIdleConnections/context cancellation.
6. sreya

   closed this as completedin coder#22292on Feb 25, 2026
7. pull

   added a commit that references this issue on Feb 25, 2026

   fix: use separate http.Transports for wsproxy tests (coder#22292)

   ...

   4f34452