Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[ci] Splits Jest integration tests #125454

Merged
merged 2 commits into from
Feb 14, 2022
Merged

[ci] Splits Jest integration tests #125454

merged 2 commits into from
Feb 14, 2022

Conversation

tylersmalley
Copy link
Contributor

@tylersmalley tylersmalley commented Feb 12, 2022 •
edited
Loading

We need to split the Jest integration tests on CI as it's approaching the current CI duration. This refactors the check for Jest config files to also handle Jest integration tests, and to be more efficient (no longer uses jest --listTests). The next step is to split the groups based on the execution time.

Used that script to generate the necessary Jest integration config files

node scripts/check_jest_configs.js --fix

@tylersmalley tylersmalley force-pushed the split-jest-integration branch 5 times, most recently from 5c823c0 to 0b8cca9 Compare February 12, 2022 21:44
[ci] Splits Jest integrtion tests
1f7ac5d 
Signed-off-by: Tyler Smalley <tyler.smalley@elastic.co>
@tylersmalley tylersmalley force-pushed the split-jest-integration branch from 0b8cca9 to 1f7ac5d Compare February 12, 2022 21:45
@tylersmalley tylersmalley changed the title [ci] Splits Jest integrtion tests [ci] Splits Jest integration tests Feb 12, 2022
@tylersmalley tylersmalley added Team:Operations Team label for Operations Team v8.0.1 v8.1.0 v8.2.0 release_note:skip Skip the PR/issue when compiling release notes v7.16.4 v7.17.1 auto-backport Deprecated - use backport:version if exact versions are needed and removed v7.16.4 labels Feb 13, 2022
@tylersmalley tylersmalley marked this pull request as ready for review February 13, 2022 00:49
@tylersmalley tylersmalley requested review from a team as code owners February 13, 2022 00:49
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-operations (Team:Operations)

@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@botelastic botelastic bot added the Team:Fleet Team label for Observability Data Collection Fleet team label Feb 14, 2022
@elasticmachine
Copy link
Contributor

Pinging @elastic/fleet (Team:Fleet)

juliaElastic
juliaElastic approved these changes Feb 14, 2022
View reviewed changes
Copy link
Contributor

@juliaElastic juliaElastic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fleet change LGTM

ymao1
ymao1 approved these changes Feb 14, 2022
View reviewed changes
Copy link
Contributor

@ymao1 ymao1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Response Ops changes LGTM

@tylersmalley tylersmalley enabled auto-merge (squash) February 14, 2022 16:06
@tylersmalley tylersmalley merged commit 435b772 into elastic:main Feb 14, 2022
@tylersmalley tylersmalley deleted the split-jest-integration branch February 14, 2022 16:12
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Feb 14, 2022
@kibanamachine
[ci] Splits Jest integration tests (elastic#125454)
1fad3e5 
(cherry picked from commit 435b772)
@kibanamachine kibanamachine mentioned this pull request Feb 14, 2022
Merged
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Feb 14, 2022
@kibanamachine
[ci] Splits Jest integration tests (elastic#125454)
f0498a0 
(cherry picked from commit 435b772)
@kibanamachine kibanamachine mentioned this pull request Feb 14, 2022
Merged
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Feb 14, 2022
@kibanamachine
[ci] Splits Jest integration tests (elastic#125454)
95eae07 
(cherry picked from commit 435b772)
@kibanamachine kibanamachine mentioned this pull request Feb 14, 2022
Merged
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
✅ 8.1
✅ 8.0
✅ 7.17

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Feb 14, 2022
@kibanamachine
[ci] Splits Jest integration tests (#125454) (#125533)
25f4f43 
(cherry picked from commit 435b772)

Co-authored-by: Tyler Smalley <tyler.smalley@elastic.co>
kibanamachine added a commit that referenced this pull request Feb 14, 2022
@kibanamachine
[ci] Splits Jest integration tests (#125454) (#125532)
87895d8 
(cherry picked from commit 435b772)

Co-authored-by: Tyler Smalley <tyler.smalley@elastic.co>
kibanamachine added a commit that referenced this pull request Feb 14, 2022
@kibanamachine
[ci] Splits Jest integration tests (#125454) (#125531)
654a63c 
(cherry picked from commit 435b772)

Co-authored-by: Tyler Smalley <tyler.smalley@elastic.co>
@tylersmalley tylersmalley mentioned this pull request Feb 15, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@juliaElastic juliaElastic juliaElastic approved these changes

@vadimkibana vadimkibana vadimkibana approved these changes

@ymao1 ymao1 ymao1 approved these changes

@jbudz jbudz jbudz approved these changes

@pgayvallet pgayvallet pgayvallet approved these changes

Assignees
No one assigned
Labels
auto-backport Deprecated - use backport:version if exact versions are needed release_note:skip Skip the PR/issue when compiling release notes Team:Fleet Team label for Observability Data Collection Fleet team Team:Operations Team label for Operations Team v7.17.1 v8.0.1 v8.1.0 v8.2.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
9 participants
@tylersmalley @elasticmachine @kibana-ci @kibanamachine @pgayvallet @jbudz @ymao1 @vadimkibana @juliaElastic