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 localPV workload dashboard #29

Merged
merged 14 commits into from
Jun 28, 2021

Conversation

rajaSahil
Copy link
Member

@rajaSahil rajaSahil commented Jun 10, 2021

Signed-off-by: Sahil Raja [email protected]

What this PR does?

  • Add dashboard to monitor localPV workload stats.
  • Will add service and servicemonitor in the next PR.

NOTE

  • These dashboards are built with the help of grafana variables.

To map persistentvolume_claim with localPV device using join, use this expression

  • kube_persistentvolumeclaim_info * on(volumename) group_left openebs_size_of_volume

To map workload with localPV device using join, use this expression

  • openebs_size_of_volume * on(volumename) group_right(device) kube_persistentvolumeclaim_info * on(persistentvolumeclaim) group_left kube_pod_spec_volumes_persistentvolumeclaims_info * on(pod) group_left namespace_workload_pod:kube_pod_owner:relabel

How dashboards mapping are done?

  • To map workload to mountPoint for filesystem dashboards:

    • Metrics hierarchy
      Screenshot from 2021-06-28 09-33-43
  • To map workload to localPV device for disk dashboards:

    • Metrics hierarchy
      Screenshot from 2021-06-28 09-28-30

Variable dependencies:

Screenshot from 2021-06-28 08-46-36

Screenshots

Screenshot from 2021-06-14 16-19-36
Screenshot from 2021-06-12 08-30-20
Screenshot from 2021-06-12 08-12-27
Screenshot from 2021-06-12 08-12-36
Screenshot from 2021-06-12 08-12-49
Screenshot from 2021-06-12 08-12-59
Screenshot from 2021-06-12 08-13-09
Screenshot from 2021-06-12 08-13-21

Note

Ref

@rajaSahil rajaSahil changed the title Add workload dashboard Add LVM-localPV workload dashboard Jun 10, 2021
@rajaSahil rajaSahil marked this pull request as ready for review June 12, 2021 02:45
@rajaSahil rajaSahil self-assigned this Jun 12, 2021
@kmova kmova requested a review from iyashu June 12, 2021 11:08
@rajaSahil rajaSahil force-pushed the Add-Workload-Dashboard branch from 7d1e9bc to f3034da Compare June 14, 2021 10:53
@rajaSahil rajaSahil force-pushed the Add-Workload-Dashboard branch from f3034da to fd72355 Compare June 16, 2021 09:34
@iyashu
Copy link

iyashu commented Jun 16, 2021

@kmova @rajaSahil Is it possible to directly import the dashboard json in grafana without doing helm install? I was trying to add above dashboard in my local cluster, but there seems to be some helm template vars being used in dashboard json as well.

Signed-off-by: Sahil Raja <[email protected]>
@rajaSahil
Copy link
Member Author

@iyashu There are no helm template vars used in the json. However, there were few links used that might have caused the error. I have removed those links. So can you please try again to import the updated json in grafana?

@rajaSahil rajaSahil changed the title Add LVM-localPV workload dashboard Add localPV workload dashboard Jun 28, 2021
Signed-off-by: Sahil Raja <[email protected]>
Signed-off-by: Sahil Raja <[email protected]>
Copy link
Member

@avishnu avishnu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Copy link
Member

@Ab-hishek Ab-hishek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@avishnu avishnu merged commit 6a1e480 into openebs:develop Jun 28, 2021
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

Successfully merging this pull request may close these issues.

4 participants