github-mirror/n8n
1
0
Fork 0
mirror of https://github.com/n8n-io/n8n.git synced 2026-05-12 16:10:30 +02:00
Code Issues Actions 15 Packages Projects Releases Wiki Activity

ci: Enable Ryuk cleanup for testcontainers (#25848)

Browse Source 
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Declan Carroll 2026-02-19 07:27:03 +00:00 committed by GitHub
parent a2034d8f42
commit 343d2af726
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
3 changed files with 3 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
.github/workflows/test-e2e-coverage-weekly.yml vendored
View File

@ -7,7 +7,6 @@ on:
env:
NODE_OPTIONS: --max-old-space-size=16384
TESTCONTAINERS_RYUK_DISABLED: true
PLAYWRIGHT_WORKERS: 4
PLAYWRIGHT_BROWSERS_PATH: packages/testing/playwright/.playwright-browsers

2
.github/workflows/test-e2e-reusable.yml vendored
View File

@ -80,8 +80,6 @@ on:
env:
NODE_OPTIONS: ${{ contains(inputs.runner, '2vcpu') && '--max-old-space-size=6144' || '' }}
# Disable Ryuk to avoid issues with Docker since it needs privileged access, containers are cleaned on teardown anyway
TESTCONTAINERS_RYUK_DISABLED: true
PLAYWRIGHT_WORKERS: ${{ inputs.workers != '' && inputs.workers || '2' }}
# Browser cache location - must match install-browsers script
PLAYWRIGHT_BROWSERS_PATH: packages/testing/playwright/.playwright-browsers

7
packages/cli/jest.config.integration.testcontainers.js
View File

@ -4,10 +4,9 @@
*
* Usage: pnpm test:postgres:tc
*
* Note: TESTCONTAINERS_RYUK_DISABLED=true is set in the npm script because:
* 1. Ryuk requires privileged Docker access not available in all CI environments
* 2. Containers use withReuse() and are cleaned up in globalTeardown
* 3. If tests crash, containers may linger - run `docker ps` to check
* Note: Ryuk is enabled by default and handles container cleanup on process exit,
* including crashes and timeouts. Containers also use withReuse() and are
* cleaned up in globalTeardown as a secondary cleanup mechanism.
*/
/** @type {import('jest').Config} */