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

UI Metrics use findAll to retrieve all Saved Objects #59891

Merged
merged 3 commits into from
Mar 12, 2020

Conversation

afharo
Copy link
Member

@afharo afharo commented Mar 11, 2020

Summary

Change the UI Metrics UsageCollection fetcher to use findAll to paginate through all the Saved Objects stored when reporting to telemetry.

Fixes #59883

Checklist

Delete any items that are not applicable to this PR.

For maintainers

@elasticmachine
Copy link
Contributor

Pinging @elastic/pulse (Team:Pulse)

@afharo
Copy link
Member Author

afharo commented Mar 11, 2020

@elasticmachine merge upstream

@afharo
Copy link
Member Author

afharo commented Mar 11, 2020

/cc @elastic/kibana-platform maybe they think it's a good idea to provide a findAll method in the SavedObjects client to avoid every plugin creating its own paginator?

Copy link
Contributor

@TinaHeiligers TinaHeiligers left a comment

Choose a reason for hiding this comment

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

There are some copy-paste errors in index.test.ts. Other than that, nice use of recursion and thanks for cleaning up the use of any in telemetry_ui_metric_collector!

Co-Authored-By: Christiane (Tina) Heiligers <[email protected]>
@afharo afharo requested a review from TinaHeiligers March 12, 2020 09:43
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

History

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

Copy link
Member

@Bamieh Bamieh left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@TinaHeiligers TinaHeiligers left a comment

Choose a reason for hiding this comment

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

LGTM!

@afharo afharo merged commit 3be4573 into elastic:master Mar 12, 2020
@afharo afharo deleted the telemetry/ui_metrics-findAll branch March 12, 2020 16:42
afharo added a commit to afharo/kibana that referenced this pull request Mar 12, 2020
* UI Metrics use findAll to retrieve all Saved Objects

* Rename test description

Co-Authored-By: Christiane (Tina) Heiligers <[email protected]>

Co-authored-by: Elastic Machine <[email protected]>
Co-authored-by: Christiane (Tina) Heiligers <[email protected]>
gmmorris added a commit to gmmorris/kibana that referenced this pull request Mar 12, 2020
* master: (45 commits)
  skip flaky suite (elastic#59717)
  UI Metrics use findAll to retrieve all Saved Objects (elastic#59891)
  [Discover] Migrate Context mocha tests to use Jest (elastic#59658)
  [Maps] Move redux reducers and store logic to NP (elastic#58294)
  rebalance x-pack groups (elastic#58930)
  [Discover] Reimplement $route.reload when index pattern changes (elastic#59877)
  [Upgrade Assistant Meta] Breaking changes issue template (elastic#59745)
  Skip CI based on changes in PR (elastic#59939)
  [ML] Transforms: Replace KqlFilterBar with QueryStringInput. (elastic#59723)
  [ML] Functional tests - stabilize date_nanos test (elastic#59986)
  [ML] Typescripting client side endpoint functions (elastic#59928)
  a11y tests on adding columns to discover table (elastic#59375)
  fix graph plugin config path (elastic#59540)
  fix vega config issues (elastic#59737)
  [Upgrade Assistant] Open And Close Slight Refactor (elastic#59890)
  [ML] Adding shared services to ml setup contract (elastic#59730)
  [Visualize] Fix linked search behavior (elastic#59690)
  [ML] Register NP ML plugin for Kibana management section. (elastic#59762)
  [Lens] Adds using queries/filters for field existence endpoint (elastic#59033)
  Delete FilterStateManager and QueryFilter :-D (elastic#59872)
  ...
afharo added a commit that referenced this pull request Mar 13, 2020
* UI Metrics use findAll to retrieve all Saved Objects

* Rename test description

Co-Authored-By: Christiane (Tina) Heiligers <[email protected]>

Co-authored-by: Elastic Machine <[email protected]>
Co-authored-by: Christiane (Tina) Heiligers <[email protected]>

Co-authored-by: Elastic Machine <[email protected]>
Co-authored-by: Christiane (Tina) Heiligers <[email protected]>
@lukeelmers lukeelmers added Feature:Telemetry Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc labels Oct 1, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-core (Team:Core)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Telemetry release_note:fix Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc v7.7.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Telemetry] Find All Saved Object Entries for ui_metrics
6 participants