- Notifications
You must be signed in to change notification settings - Fork928
Open
Description
PR#13134 added terraform provisioning to e2e tests, which is currently skipped:
coder/site/e2e/tests/createWorkspace.spec.ts
Lines 153 to 157 in3173a5e
| test("create docker workspace",async({ page})=>{ | |
| test.skip( | |
| true, | |
| "creating docker containers is currently leaky. They are not cleaned up when the tests are over.", | |
| ); |
Any docker containers created are not cleaned up. Until we can ensure tests do not leak containers, any terraform tests will be disaled.
Idea
- Add a label to all containers created in the e2e tests. In an
afterAllhook (any maybebeforeAlltoo) delete all containers with said label. - Find some way to force the container to die after
Xminutes with--rmand something like overriding the entry point withtimeout 10m # ... default entry point