-
Notifications
You must be signed in to change notification settings - Fork 435
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
[O11y][Apache] Migrate Metrics overview
dashboard visualizations to lens in kibana 8.3.0
#5636
[O11y][Apache] Migrate Metrics overview
dashboard visualizations to lens in kibana 8.3.0
#5636
Conversation
/test |
🌐 Coverage report
|
…into lens_migration-apache-metrics-8.3.0 Conflicts: packages/apache/changelog.yml packages/apache/kibana/visualization/apache-22057f20-3a12-11eb-8946-296aab7b13db.json packages/apache/kibana/visualization/apache-47820ce0-3a1d-11eb-8946-296aab7b13db.json packages/apache/kibana/visualization/apache-99666080-3a20-11eb-8946-296aab7b13db.json packages/apache/manifest.yml
As discussed with @SubhrataK,
|
…into lens_migration-apache-metrics-8.3.0
@agithomas Updated the PR based on this comment. Thanks! |
7abe485
to
8777ad0
Compare
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.
LGTM!
@SubhrataK , can you please confirm if the new way of representation is good ? Please refer the screenshot |
@agithomas The line split charts are basically splitting charts into 5 or 10 line charts as per provided field. We don't have any functionality to migrate this kind of chart in lens. Also as discuss with @SubhrataK, the splitting chart has poor readability. Hence we have converted to table chart in which user can see their actual values based on time interval. Please refer the screenshot for more information. |
As discuss with @SubhrataK, we are going to rename @agithomas If you feel like there are many fields in the table feel free to suggest any fields that we can remove. Field names that have used |
My concern is mainly around representing the CPU usage in tabular manner, when a line graph could explain the trends much better. |
@SubhrataK / @agithomas We have converted the line split chart to lens line chart. Let me know if it looks good to you so that we can update the PR accordingly. Please refer this screenshot. |
This looks much better and user friendly. Hope you can update the screenshot. |
@agithomas Updated the PR with suggested changes. Thanks! |
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.
LGTM!
Package apache - 1.9.0 containing this change is available at https://epr.elastic.co/search?package=apache |
What does this PR do?
Migrate visualizations to lens for Apache integration package
As per discussion with @SubhrataK,
Total accesses
, Kept the original value (not thek
) in this case, as the number of accesses has to be the exact value and not a rounded off version.Statistics for Apache Lens migration:
Checklist
changelog.yml
file.Author's checklist
Related issues
Issues Identified
Note:
[Metrics Apache] Overview
dashboard, because the current screenshot is not updated.