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

[O11y][Vsphere] Lens Migration #5676

Closed
6 tasks done
rajvi-patel-22 opened this issue Mar 24, 2023 · 9 comments
Closed
6 tasks done

[O11y][Vsphere] Lens Migration #5676

rajvi-patel-22 opened this issue Mar 24, 2023 · 9 comments
Assignees

Comments

@rajvi-patel-22
Copy link
Contributor

rajvi-patel-22 commented Mar 24, 2023

Manually migrate Vsphere visualizations to lens in current Kibana version 8.7.0 itself.

Preparation of data for testing

  • Use sample_event.json / setup live instance using docker / mock data using mock server to populate dashboards

Migration stats

VSphere dashboard Before Migration   After Migration  
  Lens Visualization Lens Visualization
[Metrics VSphere] Hosts Overview 0 7 7 0
[Metrics VSphere] VMs overview 0 5 5 0
  0 12 12 0

Dashboard : [Metrics VSphere] Hosts Overview

Dashboard : [Metrics VSphere] VMs overview

Verification and Validation

  • Verification of functionality remains the same after migration in Kibana
  • Verification of data count of after-migrated visualization with before-migrated visualization in Kibana
  • Verification of Dashboard are not distorted in supported Kibana version (8.7.0)

Issues Identified

@rajvi-patel-22
Copy link
Contributor Author

@agithomas @SubhrataK

There is one issue in Kibana v8.5.0. The new metric visualization doesn't allow to add a panel title which is going to be fixed in Kibana 8.7+. Because of this issue, the panel title of new metric visualization is not shown in v8.5.0

GitHub Issue: elastic/kibana#148563

Below is the screenshot of panels before and after migration. The panel title Host top RAM util [Metrics VSphere], Host top CPU util [Metrics VSphere] and Top Datastore Used [Metrics VSphere] are not visible after migration. However, it will be visible in the future version once this issue will be merged. (Tested on snapshot-8.7.0 and panel titles are visible)

Before migration:
vsphere_dashboard (1)

After migration:
vsphere_dashboard

Should we keep minimum Kibana version to 8.5.0 or should we upgrade it to 8.7.0?

@agithomas
Copy link
Contributor

@lalit-satapathy , @ishleenk17 , are there any concerns?

@ishleenk17
Copy link
Contributor

The new metric visualization doesn't allow to add a panel title which is going to be fixed in Kibana 8.7+

@rajvi-elastic : Having panel names is important for the user to understand what the panel is about. So it should be present.
I suppose this would be the issue with multiple integrations. How are we handling there?

Also, for the issue identified "Support gauge chart", do we have ETA on when this will be resolved ?

@rajvi-patel-22
Copy link
Contributor Author

@ishleenk17, in other visualizations, title is same as the label (as you can see in the below screenshot). And as per our best practices, we are disabling the title in metric visualizations if the title and labels are same.
227493012-d9e67cfa-1852-4721-9535-6e279dc96327
227493095-1efcd7d6-4b52-4fde-9ede-aff39c851b07

The issue that you are talking about has been closed already. And as discussed with @SubhrataK we are migrating gauge visualizations to metric or horizontal/vertical gauge chart.

@ishleenk17
Copy link
Contributor

ishleenk17 commented Apr 5, 2023

@rajvi-elastic, We can update the kibana version to 8.7.

@rajvi-patel-22
Copy link
Contributor Author

@rajvi-elastic, We can update the kibana version to 8.7.

As discussed with @ishleenk17, this issue has been fixed in 8.7.0 Kibana version. Hence, We'll keep minimum Kibana version to 8.7.0 .

@agithomas
Copy link
Contributor

@rajvi-elastic , the PR link seem to be broken in the description. Please update.

@rajvi-patel-22
Copy link
Contributor Author

@rajvi-elastic , the PR link seem to be broken in the description. Please update.

@agithomas I have updated the links.

@rajvi-patel-22
Copy link
Contributor Author

PR is approved and merged, hence closing this issue:
#5810
#5811

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants