-
Notifications
You must be signed in to change notification settings - Fork 52.2k
ci: Improves CI test performance #23189
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
Conversation
Enhances CI test performance by sharding frontend tests across more workers, switching to smaller VMs, and increasing worker memory limits. Also configures vitest to use threads. CAT-1964
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No issues found across 4 files
This comment has been minimized.
This comment has been minimized.
|
E2E Tests: n8n tests passed after 7m 14.9s Run Details
Groups
This message was posted automatically by
currents.dev | Integration Settings
|
alexgrozav
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
|
Got released with |
Summary
There was a change in March where we set the memory limits from 1024 (1GB) -> 1MB (1024 sort of)
Related Linear tickets, Github issues, and Community forum posts
https://linear.app/n8n/issue/CAT-1964/ci-optimize-jest-and-vitest-test-runner-performance
Review / Merge checklist
release/backport(if the PR is an urgent fix that needs to be backported)