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

Add telemetry to vis_type_xy plugin #82493

Closed
nickofthyme opened this issue Nov 3, 2020 · 4 comments
Closed

Add telemetry to vis_type_xy plugin #82493

nickofthyme opened this issue Nov 3, 2020 · 4 comments
Assignees
Labels
Feature:Telemetry Feature:XYAxis XY-Axis charts (bar, area, line) Team:Visualizations Visualization editors, elastic-charts and infrastructure

Comments

@nickofthyme
Copy link
Contributor

nickofthyme commented Nov 3, 2020

Describe the feature:
Add telemetry to vis_type_xy plugin (See #46599)

Metrics include:

  • All telemetry being collected from vis_type_vislib, if any.
  • Usage of all new editor options. Currently only detailedTooltip and fittingFunction .
  • Users with advanced setting (visualization:visualize:chartsLibrary) enabled.

Describe a specific use case for the feature:
To capture data about usage of newChartUi

@nickofthyme nickofthyme added Feature:XYAxis XY-Axis charts (bar, area, line) Feature:Telemetry Team:Visualizations Visualization editors, elastic-charts and infrastructure labels Nov 3, 2020
@nickofthyme nickofthyme self-assigned this Nov 3, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-app (Team:KibanaApp)

@VladLasitsa
Copy link
Contributor

@nickofthyme, You have added as a example the similar task for Lens. As I saw for Lens was created structure for aggregating UI events. But we have usageCollection plugin which we use in Discover for this goal. What way we should use?
The second question which metrics we should collect.

@nickofthyme
Copy link
Contributor Author

Hey @VladLasitsa,

To be honest, I'm not exactly sure how this should be implemented. I just listed the Lens issue for reference on a way it could be handled.

As far as what is to be collected. I updated the description with a list. If I think of any more I will update the issue.

@stratoula
Copy link
Contributor

This collector https://github.com/elastic/kibana/blob/18f7f042c1b6bd36c0cf09c9fed4396e7484e0a4/src/plugins/kibana_usage_collection/server/collectors/management/telemetry_management_collector.ts#L54 reports all the advances settings that are different from the default. So no need to do something about it from our side. What is missing is the mapping of the new field that I have asked it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Telemetry Feature:XYAxis XY-Axis charts (bar, area, line) Team:Visualizations Visualization editors, elastic-charts and infrastructure
Projects
None yet
Development

No branches or pull requests

4 participants