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.

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

[ftr] split serverless config to speedup test run on CI #168195

Merged
merged 7 commits into from
Oct 6, 2023

Conversation

dmlemeshko
Copy link
Member

@dmlemeshko dmlemeshko commented Oct 6, 2023

Summary

After 164798 was merged CI singnificantly slowed down because f these FTR groups:

x-pack/test_serverless/functional/test_suites/security/common_configs/config.group2.ts: 75.6 minutes
x-pack/test_serverless/functional/test_suites/search/common_configs/config.group2.ts: 75 minutes
x-pack/test_serverless/functional/test_suites/observability/common_configs/config.group2.ts: 74.7 minutes

This PR splits each group into 3.

@dmlemeshko dmlemeshko requested a review from pheyos October 6, 2023 10:05
@dmlemeshko dmlemeshko marked this pull request as ready for review October 6, 2023 16:30
@dmlemeshko dmlemeshko requested review from a team as code owners October 6, 2023 16:30
@dmlemeshko dmlemeshko added release_note:skip Skip the PR/issue when compiling release notes v8.12.0 v8.11.1 labels Oct 6, 2023
@dmlemeshko dmlemeshko enabled auto-merge (squash) October 6, 2023 16:31
Copy link
Contributor

@drewdaemon drewdaemon left a comment

Choose a reason for hiding this comment

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

Visualizations changes LGTM

Copy link
Member

@pheyos pheyos left a comment

Choose a reason for hiding this comment

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

LGTM

@kibana-ci
Copy link
Collaborator

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #73 / detection engine api security and spaces enabled - rule execution logic ES|QL rule type timestamp override and fallback should generate the correct alerts when timestamp_override defined

Metrics [docs]

✅ unchanged

History

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

@dmlemeshko dmlemeshko merged commit d05349b into elastic:main Oct 6, 2023
@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
8.11 Backport failed because of merge conflicts

Manual backport

To create the backport manually run:

node scripts/backport --pr 168195

Questions ?

Please refer to the Backport tool documentation

dej611 pushed a commit to dej611/kibana that referenced this pull request Oct 17, 2023
## Summary

After [164798](elastic#164798) was merged
CI singnificantly slowed down because f these FTR groups:

```
x-pack/test_serverless/functional/test_suites/security/common_configs/config.group2.ts: 75.6 minutes
x-pack/test_serverless/functional/test_suites/search/common_configs/config.group2.ts: 75 minutes
x-pack/test_serverless/functional/test_suites/observability/common_configs/config.group2.ts: 74.7 minutes
```

This PR splits each group into 3.
@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Jun 2, 2024
@kibanamachine
Copy link
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add the label auto-backport or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 168195 locally

@nickofthyme nickofthyme added the backport:skip This commit does not require backporting label Jun 3, 2024
@kibanamachine kibanamachine removed the backport missing Added to PRs automatically when the are determined to be missing a backport. label Jun 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting release_note:skip Skip the PR/issue when compiling release notes v8.11.1 v8.12.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants