Skip to content

test(coderd): fix TestWorkspaceWatcher hang waiting for update #8992

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

Merged
merged 1 commit into from
Aug 9, 2023

Conversation

spikecurtis
Copy link
Contributor

Fixes #8968

#8584 was an incomplete fix because a call to wait() consumes an update, so if we skip pending and go straight to Failed, we can't wait for another update, as this just times out.

Signed-off-by: Spike Curtis <spike@coder.com>
@spikecurtis spikecurtis merged commit e0f644c into main Aug 9, 2023
@spikecurtis spikecurtis deleted the spike/8968-workspace-watcher-flake branch August 9, 2023 12:09
@github-actions github-actions bot locked and limited conversation to collaborators Aug 9, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

test flake: coderd TestWorkspaceWatcher
2 participants