-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
[Telemetry] Add canvas stats to Legacy telemetry bulk uploader #21103
[Telemetry] Add canvas stats to Legacy telemetry bulk uploader #21103
Conversation
💔 Build Failed |
💚 Build Succeeded |
💚 Build Succeeded |
@@ -8,13 +8,15 @@ import { get, set, merge } from 'lodash'; | |||
import { | |||
KIBANA_SYSTEM_ID, | |||
LOGSTASH_SYSTEM_ID, | |||
CANVAS_SYSTEM_ID, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changing this file should not have to be necessary
@@ -0,0 +1,62 @@ | |||
/* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There isn't really anything "canvas" specific in this file. It was copied from the reporting file. Seems like it could be made generic at a higher level
I'm just going to go ahead and close this, because we'd rather have #21239 |
Telemetry uses an internal Bulk Uploader, which is part of
x-pack/plugins/monitoring/server/kibana_monitoring
to collect and upload stats to Elasticsearch in a method calledcombineStatsLegacy
. This is legacy because the long-term goal is to have collection done externally through Metricbeat.This PR adds Canvas stats to the Bulk Uploader's payload data.