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

Reporting usage collector fetch migration #86675

Merged
merged 2 commits into from
Dec 22, 2020
Merged

Reporting usage collector fetch migration #86675

merged 2 commits into from
Dec 22, 2020

Conversation

TinaHeiligers
Copy link
Contributor

Summary

Uses the new elasticsearch client in the reporting usage collector's fetch implementation.
Part of #86358
.

Checklist

Delete any items that are not applicable to this PR.

For maintainers

@TinaHeiligers TinaHeiligers added (Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead Feature:Telemetry v8.0.0 release_note:skip Skip the PR/issue when compiling release notes Team:Reporting Services v7.12.0 labels Dec 21, 2020
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Distributable file count

id before after diff
default 47142 47902 +760

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

@TinaHeiligers TinaHeiligers marked this pull request as ready for review December 21, 2020 21:21
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-reporting-services (Team:Reporting Services)

tsullivan
tsullivan approved these changes Dec 22, 2020
View reviewed changes
Copy link
Member

@tsullivan tsullivan left a comment

Choose a reason for hiding this comment

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

LGTM!

@TinaHeiligers TinaHeiligers merged commit a98550b into elastic:master Dec 22, 2020
@TinaHeiligers TinaHeiligers deleted the reporting-usage-collector-fetch-migration branch December 22, 2020 18:23
@TinaHeiligers TinaHeiligers mentioned this pull request Dec 22, 2020
Merged
TinaHeiligers added a commit that referenced this pull request Dec 22, 2020
@TinaHeiligers
[7.x] Reporting usage collector fetch migration (#86675) (#86818)
f93622e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tsullivan tsullivan tsullivan approved these changes

Assignees
No one assigned
Labels
(Deprecated) Feature:Reporting Use Reporting:Screenshot, Reporting:CSV, or Reporting:Framework instead Feature:Telemetry release_note:skip Skip the PR/issue when compiling release notes v7.12.0 v8.0.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@TinaHeiligers @kibanamachine @elasticmachine @tsullivan