- Notifications
You must be signed in to change notification settings - Fork1k
Commit8282393
authored
fix: Bump up workspace history timeout (#149)
This is an attempted fix for failures like:https://github.com/coder/coder/runs/5043435263?check_suite_focus=true#step:7:32Looking at the timing of the test:``` t.go:56: 2022-02-02 21:33:21.964 [DEBUG](terraform-provisioner)<provision.go:139>ran apply t.go:56: 2022-02-02 21:33:21.991 [DEBUG](provisionerd)<provisionerd.go:162>skipping acquire; job is already running t.go:56: 2022-02-02 21:33:22.050 [DEBUG](provisionerd)<provisionerd.go:162>skipping acquire; job is already running t.go:56: 2022-02-02 21:33:22.090 [DEBUG](provisionerd)<provisionerd.go:162>skipping acquire; job is already running t.go:56: 2022-02-02 21:33:22.140 [DEBUG](provisionerd)<provisionerd.go:162>skipping acquire; job is already running t.go:56: 2022-02-02 21:33:22.195 [DEBUG](provisionerd)<provisionerd.go:162>skipping acquire; job is already running t.go:56: 2022-02-02 21:33:22.240 [DEBUG](provisionerd)<provisionerd.go:162>skipping acquire; job is already running workspacehistory_test.go:122: Error Trace:workspacehistory_test.go:122 Error: Condition never satisfied Test: TestWorkspaceHistory/CreateHistory```It appears that the `terraform apply` job had just finished - with less than a second to spare until our `require.Eventually` completes - but there's still work to be done (ie, collecting the state files). So my suspicion is that terraform might, in some cases, exceed our 5s timeout.Note that in the setup for this test - there is a similar project history wait that waits for 15s, so I borrowed that here.In the future - we can look at potentially using a simple echo provider to exercise this in the unit test, in a way that is more reliable in terms of timing. I'll log an issue to track that.1 parent9cc4756 commit8282393
1 file changed
+1
-1
lines changedOriginal file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
123 | 123 |
| |
124 | 124 |
| |
125 | 125 |
| |
126 |
| - | |
| 126 | + | |
127 | 127 |
| |
128 | 128 |
| |
129 | 129 |
| |
|
0 commit comments
Comments
(0)