Skip to content

Conversation

@spalger
Copy link
Contributor

@spalger spalger commented Feb 5, 2021

Rather than shipping metrics to ci-stats during the build, this PR writes the metrics to the filesystem so that it can be cached, and then shipped to ci-stats as a separate step so that we can work on caching the build step with bazel.

  • writes metrics to metrics.json files in each bundle from the optimizer
  • collects the metrics into the target/optimizer_bundle_metrics.json after running the optimizer in the build BuildKibanaPlatformPlugins task of the build script
  • ships the combined metrics to ci-stats with node scripts/ship_ci_stats --metrics target/optimizer_bundle_metrics.json after running node scripts/build in the places where we want metrics.

@spalger spalger added release_note:skip Skip the PR/issue when compiling release notes Team:Operations Kibana-Operations Team v7.12.0 v8.0.0 labels Feb 6, 2021
@spalger spalger marked this pull request as ready for review February 6, 2021 00:19
@spalger spalger requested a review from a team as a code owner February 6, 2021 00:19
@elasticmachine
Copy link
Contributor

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

@spalger spalger requested a review from mistic February 6, 2021 00:20
Copy link
Member

@mistic mistic left a comment

Choose a reason for hiding this comment

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

LGTM 🎉

@spalger

This comment has been minimized.

@mistic
Copy link
Member

mistic commented Feb 6, 2021

@elasticmachine merge upstream

@spalger
Copy link
Contributor Author

spalger commented Feb 8, 2021

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

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

@spalger spalger merged commit a0ce7b5 into elastic:master Feb 8, 2021
@spalger spalger deleted the implement/separate-report-tasks branch February 8, 2021 17:47
spalger added a commit that referenced this pull request Feb 8, 2021
) (#90659)

Co-authored-by: spalger <spalger@users.noreply.github.com>
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release_note:skip Skip the PR/issue when compiling release notes Team:Operations Kibana-Operations Team v7.12.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants