After a rebase on main, the test is now failing due to the dialer creating an error log while the server is down:

    t.go:115: 2024-11-19 01:50:26.405 [erro]  client: failed to dial tailnet v2+ API  error="failed to WebSocket dial: expected handshake response status code 101 but got 502" ...
        sdk_err= GET http://127.0.0.1:46789/api/v2/workspaceagents/54909b8e-b909-46fa-a33d-33afa7780731/coordinate?resume_token=*** unexpected status code 502: unexpected non-JSON response "text/plain; charset=utf-8"
                 	Error: server is not started
         *** slogtest: log detected at level ERROR; TEST FAILURE ***
    t.go:115: 2024-11-19 01:50:26.405 [erro]  client: failed to dial tailnet v2+ API  error="failed to WebSocket dial: expected handshake response status code 101 but got 502"
         *** slogtest: log detected at level ERROR; TEST FAILURE ***

After a rebase on main, the test is now failing due to the dialer creating an error log while the server is down:

    t.go:115: 2024-11-19 01:50:26.405 [erro]  client: failed to dial tailnet v2+ API  error="failed to WebSocket dial: expected handshake response status code 101 but got 502" ...
        sdk_err= GET http://127.0.0.1:46789/api/v2/workspaceagents/54909b8e-b909-46fa-a33d-33afa7780731/coordinate?resume_token=*** unexpected status code 502: unexpected non-JSON response "text/plain; charset=utf-8"
                 	Error: server is not started
         *** slogtest: log detected at level ERROR; TEST FAILURE ***
    t.go:115: 2024-11-19 01:50:26.405 [erro]  client: failed to dial tailnet v2+ API  error="failed to WebSocket dial: expected handshake response status code 101 but got 502"
         *** slogtest: log detected at level ERROR; TEST FAILURE ***

Those are real errors and we don't want to suppress them. You can configure the logger not to fail the test like

logger := slogtest.Make(t, &slogtest.Options{IgnoreErrors: true}).Level(slog.LevelDebug)

Okay, I think we can finally merge this?