-
Notifications
You must be signed in to change notification settings - Fork 461
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][ZooKeeper] Lens migration for visualizations to Kibana version 8.2.0 #5691
[O11y][ZooKeeper] Lens migration for visualizations to Kibana version 8.2.0 #5691
Conversation
🌐 Coverage report
|
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. It may be best to add the unit of latency in the Y-axis for the latency panel.
If you can add the unit in the Y-axis for the latency, it will be good. If it is a time taking thing, please create a backlog ticket
Yes @agithomas. We can add the unit in Y-axis label. As per the readme file the latency is measured in milliseconds. So we will add |
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!
Please complete the peer review
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 zookeeper - 1.6.0 containing this change is available at https://epr.elastic.co/search?package=zookeeper |
What does this PR do?
Migrate visualizations to lens for ZooKeeper integration package
Statistics for ZooKeeper Lens migration:
Checklist
changelog.yml
file.Author's Checklist
Related issues
Issues Identified
The chart do not render negative values elastic-charts#135
Impact : [ Severity : Low, Workaround : This will be fixed in 8.8 ]