Skip to content

Conversation

@schmikei
Copy link
Contributor

@schmikei schmikei commented Oct 21, 2025

This PR modernizes the apache-cassandra-mixin to use commonlib and g.libsonnet to generate its dashboards and alert content.

Apache Cassandra overview

Notes: moved up the total disk usage up to the top of the dashbaord
Original conversion
image
Now:
image
image
image

Apache Cassandra nodes

image image

Apache Cassandra keyspaces

image

Apache Cassandra logs

image

@schmikei schmikei requested a review from a team as a code owner October 21, 2025 20:43
Copy link
Member

@Dasomeone Dasomeone left a comment

Choose a reason for hiding this comment

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

Couple things to fix here, but overall a good improvement for sure!

Comment on lines 77 to 78
panels.keyspacesWriteLatencyPanel { gridPos+: { w: 12 } },
panels.keyspacesReadLatencyPanel { gridPos+: { w: 12 } },
Copy link
Member

Choose a reason for hiding this comment

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

Quite possibly an argument to be made for pushing read latency up in terms of importance and being higher on the dashboard, what do you think?

Perhaps a stat panel for highest p. 95 latency on the top row?

@schmikei schmikei requested a review from Dasomeone November 3, 2025 17:08
Copy link
Member

@Dasomeone Dasomeone left a comment

Choose a reason for hiding this comment

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

Couple more comments, but almost there! Looks great :D

Copy link
Member

@Dasomeone Dasomeone left a comment

Choose a reason for hiding this comment

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

LGTM, thank you Keith!

@Dasomeone Dasomeone merged commit 38e888e into grafana:master Nov 4, 2025
11 checks passed
@schmikei schmikei deleted the feat/cassandra-modernization branch November 4, 2025 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants