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

Remove usage of Kibana plugins build in performance test pipelines #205896

Merged
merged 1 commit into from
Jan 8, 2025

Conversation

pheyos
Copy link
Member

@pheyos pheyos commented Jan 8, 2025

Summary

This PR fixes the performance test pipelines by removing the bits that rely on the plugins build.

Details

  • The plugin build has been removed with [ci] Remove plugin build #197125. Since the performance pipelines are running against a Kibana build (and not against sources), they should not need the plugin build.
  • The performance-data-set-extraction pipeline started to fail immediately after the plugin build has been removed
  • This failure went unnoticed since the scalability-benchmarking pipeline continued to work by using the last uploaded artifacts from the performance-data-set-extraction pipeline, which were available for another month. Once the old artifacts were no longer available, the scalability-benchmarking pipeline also started to fail.

@pheyos pheyos added release_note:skip Skip the PR/issue when compiling release notes v9.0.0 backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) v8.18.0 labels Jan 8, 2025
@pheyos pheyos self-assigned this Jan 8, 2025
@pheyos pheyos requested a review from a team as a code owner January 8, 2025 13:45
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #13 / CasesWebhookActionConnectorFields renders Step Validation Step 2 is properly validated

Metrics [docs]

✅ unchanged

cc @pheyos

Copy link
Member

@jbudz jbudz left a comment

Choose a reason for hiding this comment

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

Thanks for catching this, apologies for the issue.

@pheyos pheyos merged commit 9bdc995 into elastic:main Jan 8, 2025
18 checks passed
@pheyos pheyos deleted the fix_perf_pipeline branch January 8, 2025 16:15
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/12674592010

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jan 8, 2025
…lastic#205896)

## Summary

This PR fixes the performance test pipelines by removing the bits that
rely on the plugins build.

### Details

* The plugin build has been removed with elastic#197125. Since the performance
pipelines are running against a Kibana build (and not against sources),
they should not need the plugin build.
* The `performance-data-set-extraction` pipeline started to fail
immediately after the plugin build has been removed
* This failure went unnoticed since the `scalability-benchmarking`
pipeline continued to work by using the last uploaded artifacts from the
`performance-data-set-extraction` pipeline, which were available for
another month. Once the old artifacts were no longer available, the
`scalability-benchmarking` pipeline also started to fail.

(cherry picked from commit 9bdc995)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.x

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 Jan 9, 2025
…nes (#205896) (#205930)

# Backport

This will backport the following commits from `main` to `8.x`:
- [Remove usage of Kibana plugins build in performance test pipelines
(#205896)](#205896)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Robert
Oskamp","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-01-08T16:15:33Z","message":"Remove
usage of Kibana plugins build in performance test pipelines
(#205896)\n\n## Summary\r\n\r\nThis PR fixes the performance test
pipelines by removing the bits that\r\nrely on the plugins
build.\r\n\r\n### Details\r\n\r\n* The plugin build has been removed
with #197125. Since the performance\r\npipelines are running against a
Kibana build (and not against sources),\r\nthey should not need the
plugin build.\r\n* The `performance-data-set-extraction` pipeline
started to fail\r\nimmediately after the plugin build has been
removed\r\n* This failure went unnoticed since the
`scalability-benchmarking`\r\npipeline continued to work by using the
last uploaded artifacts from the\r\n`performance-data-set-extraction`
pipeline, which were available for\r\nanother month. Once the old
artifacts were no longer available, the\r\n`scalability-benchmarking`
pipeline also started to
fail.","sha":"9bdc995d3c6f1c19d242d411e039e1c9c5c6251f","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:prev-minor","v8.18.0"],"title":"Remove
usage of Kibana plugins build in performance test
pipelines","number":205896,"url":"https://github.com/elastic/kibana/pull/205896","mergeCommit":{"message":"Remove
usage of Kibana plugins build in performance test pipelines
(#205896)\n\n## Summary\r\n\r\nThis PR fixes the performance test
pipelines by removing the bits that\r\nrely on the plugins
build.\r\n\r\n### Details\r\n\r\n* The plugin build has been removed
with #197125. Since the performance\r\npipelines are running against a
Kibana build (and not against sources),\r\nthey should not need the
plugin build.\r\n* The `performance-data-set-extraction` pipeline
started to fail\r\nimmediately after the plugin build has been
removed\r\n* This failure went unnoticed since the
`scalability-benchmarking`\r\npipeline continued to work by using the
last uploaded artifacts from the\r\n`performance-data-set-extraction`
pipeline, which were available for\r\nanother month. Once the old
artifacts were no longer available, the\r\n`scalability-benchmarking`
pipeline also started to
fail.","sha":"9bdc995d3c6f1c19d242d411e039e1c9c5c6251f"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/205896","number":205896,"mergeCommit":{"message":"Remove
usage of Kibana plugins build in performance test pipelines
(#205896)\n\n## Summary\r\n\r\nThis PR fixes the performance test
pipelines by removing the bits that\r\nrely on the plugins
build.\r\n\r\n### Details\r\n\r\n* The plugin build has been removed
with #197125. Since the performance\r\npipelines are running against a
Kibana build (and not against sources),\r\nthey should not need the
plugin build.\r\n* The `performance-data-set-extraction` pipeline
started to fail\r\nimmediately after the plugin build has been
removed\r\n* This failure went unnoticed since the
`scalability-benchmarking`\r\npipeline continued to work by using the
last uploaded artifacts from the\r\n`performance-data-set-extraction`
pipeline, which were available for\r\nanother month. Once the old
artifacts were no longer available, the\r\n`scalability-benchmarking`
pipeline also started to
fail.","sha":"9bdc995d3c6f1c19d242d411e039e1c9c5c6251f"}},{"branch":"8.x","label":"v8.18.0","branchLabelMappingKey":"^v8.18.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Robert Oskamp <[email protected]>
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Jan 13, 2025
…lastic#205896)

## Summary

This PR fixes the performance test pipelines by removing the bits that
rely on the plugins build.

### Details

* The plugin build has been removed with elastic#197125. Since the performance
pipelines are running against a Kibana build (and not against sources),
they should not need the plugin build.
* The `performance-data-set-extraction` pipeline started to fail
immediately after the plugin build has been removed
* This failure went unnoticed since the `scalability-benchmarking`
pipeline continued to work by using the last uploaded artifacts from the
`performance-data-set-extraction` pipeline, which were available for
another month. Once the old artifacts were no longer available, the
`scalability-benchmarking` pipeline also started to fail.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) release_note:skip Skip the PR/issue when compiling release notes v8.18.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants