-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[ci] Run Jest tests in parallel #117188
[ci] Run Jest tests in parallel #117188
Conversation
* [ci] Run Jest tests in parallel Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co> * Disable coverage Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co> * Make hourly match prs Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co> * Update timeout Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co> * mock process.execArgv so that it is consistent * Remove comment Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co> Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com> Co-authored-by: spalger <spalger@users.noreply.github.com>
429cfea
to
73630b6
Compare
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
73630b6
to
a184ac1
Compare
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
Example of failed job with OOM: https://buildkite.com/elastic/kibana-pull-request/builds/3808#7b6cba4b-fa74-4e62-83b6-ded8c8f089f8 |
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
2d12bb1
to
e80b85e
Compare
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
e80b85e
to
87b4374
Compare
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
@elasticmachine merge upstream |
💚 Build Succeeded
Metrics [docs]
History
To update your PR or re-run it, just comment with: |
Going to merge ahead of pending reviews as the bulk of this was approved in the first attempt. The security solution team has reviewed which was the change between these two PR's. |
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
💚 Backport successful
This backport PR will be merged automatically after passing CI. |
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync. |
2 similar comments
Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync. |
Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync. |
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync. |
1 similar comment
Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync. |
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
A second attempt at #115687 after reverting due to an OOM not causing the job to fail.
This was caused by
exitCode
not being updated as it was in a subshell.security_solution/public
tests take a really long time to complete (>45 minutes). Until we have a smarter way of dividing the tests I have added a Jest config for each directory underpublic
to split it up.security_solution/sever
is unable to complete as it OOM's. The issue with this has been raised here. In the meantime, I found that splitting upserver
in the same way aspublic
gets around this issue for now.A test in
x-pack/plugins/security_solution/public/common/components/events_viewer/events_viewer.test.tsx
was making Jest exit 1, even though the tests were successful. I have opened #117586 to track re-enabling this test.