-
Notifications
You must be signed in to change notification settings - Fork 978
Open
Labels
siteArea: frontend dashboardArea: frontend dashboard
Description
PR #13134 added terraform provisioning to e2e tests, which is currently skipped:
coder/site/e2e/tests/createWorkspace.spec.ts
Lines 153 to 157 in 3173a5e
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
afterAll
hook (any maybebeforeAll
too) delete all containers with said label. - Find some way to force the container to die after
X
minutes with--rm
and something like overriding the entry point withtimeout 10m # ... default entry point
Metadata
Metadata
Assignees
Labels
siteArea: frontend dashboardArea: frontend dashboard