[Kubernetes Integration] Updating kubernetes volume dashboard

gizas commented 3 months ago

Enhancement

Proposed commit message

WHAT: Updating the Kubernetes Volume Dashboard to include Volume information per Pod
WHY: Adding more meaningful information because until now we were providing only general information for the volumes

Notes:

Volume % : Defined as Maximum value
A filter datastream.dataset=kubernetes.volume added in the visualisation

Checklist

[x] I have reviewed tips for building integrations and this pull request is aligned with them.
[x] I have verified that all data streams collect metrics or logs.
[x] I have added an entry to my package's changelog.yml file.
[x] I have verified that Kibana version constraints are current according to guidelines.

How to test this PR locally

Clone this repo
cd to integrations/build/packages/kubernetes
Build latest integaration with elastic-package build
Create a new local elastic stack with elastic-package stack up -d -v --version=8.15.0-SNAPSHOT
Install K8s integration package (latest that this manifest defines is 1.61.2)
Navigate to Kubernete Volume Dashboard

Related issues

Closes #https://github.com/elastic/beats/issues/39523

Screenshots

Screenshot 2024-06-26 at 12 13 22 PM

Screenshot 2024-06-26 at 12 13 44 PM

elasticmachine commented 3 months ago

:rocket: Benchmarks report

To see the full report comment with /test benchmark fullreport

tetianakravchenko commented 2 months ago

@gizas there is smth wrong: before this PR - there were 2 vizualisations:

after - 3 with wrong formating, one of them is a duplication:

tetianakravchenko commented 2 months ago

after clarification:

please add a title for the new vizualisation - like Volume usage [Metrics Kubernetes] (similar to other names)
clarify if pod name should be added to the Filesystem Inodes Informations [Metrics Kubernetes] and Filesystem Informations [Metrics Kubernetes] vizualisations as well

gizas commented 2 months ago

@eedugon lets clarify above questions: https://github.com/elastic/integrations/pull/10267#issuecomment-2210794216 For me the Filesystem Innformation related Visulaisations are general and should remain like that and the first visualisation for pods is enough to have the categories you want. WDYT?

I have added title Volume Usage per Pod[Metrics Kubernetes]

elasticmachine commented 2 months ago

:green_heart: Build Succeeded

Buildkite Build
Commit: dc727c62534b9cd48795d8085e832de601af2159

History

:green_heart: Build #13324 succeeded db1871cd9c76820b2fd13b9ea99c44c099d47aba
:broken_heart: Build #13316 failed 0d0204c9f3b8144e78415bdf063954009a831b6c
:green_heart: Build #12907 succeeded 6cc3a2eb59f90301ed9742b7239d607628ebda54

elastic-sonarqube[bot] commented 2 months ago

Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
98.7% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube

gizas commented 2 months ago

Synced with @eedugon offline. There was a comment to change the filter of visualisation to metricset.name but we decide not. Reason was to keep it aligned as per other dashboards that have data_stream.dataset used.

This dashboard might need another update when the type of volume will be intorduced

elasticmachine commented 2 months ago

Package kubernetes - 1.66.0 containing this change is available at https://epr.elastic.co/search?package=kubernetes

elastic / integrations