Skip to content

Conversation

@annrpom
Copy link
Contributor

@annrpom annrpom commented Nov 5, 2025

This patch adds a metric for store disk write bytes in the Storage dashboard.

Epic: none
Informs: https://github.com/cockroachlabs/support/issues/3470

Release note: None

image image image image

@cockroach-teamcity
Copy link
Member

This change is Reviewable

@annrpom annrpom requested review from a team and jbowens November 5, 2025 22:12
@annrpom annrpom marked this pull request as ready for review November 5, 2025 22:12
@annrpom annrpom requested a review from a team as a code owner November 5, 2025 22:12
@annrpom annrpom requested review from angles-n-daemons and removed request for a team November 5, 2025 22:12
Copy link
Collaborator

@jbowens jbowens 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
Collaborator

@sumeerbhola sumeerbhola left a comment

Choose a reason for hiding this comment

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

Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained (waiting on @angles-n-daemons)


pkg/ui/workspaces/db-console/src/views/cluster/containers/nodeGraphs/dashboards/storage.tsx line 486 at r1 (raw file):

        {storeMetrics(
          {
            name: "cr.store.storage.disk.write.bytes",

how about adding disk.read.bytes too.

This patch adds a metric for store disk write bytes in
the Storage dashboard.

Epic: none

Release note: None
@annrpom annrpom force-pushed the storage-dashboard-bytes branch 2 times, most recently from ce86f32 to 1928332 Compare November 6, 2025 19:56
Copy link
Contributor Author

@annrpom annrpom left a comment

Choose a reason for hiding this comment

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

Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained (waiting on @angles-n-daemons and @sumeerbhola)


pkg/ui/workspaces/db-console/src/views/cluster/containers/nodeGraphs/dashboards/storage.tsx line 486 at r1 (raw file):

Previously, sumeerbhola wrote…

how about adding disk.read.bytes too.

ok; done

This patch adds a metric for store disk read bytes in
the Storage dashboard.

Epic: none

Release note: None
@annrpom annrpom force-pushed the storage-dashboard-bytes branch from 1928332 to 8971cd2 Compare November 6, 2025 20:28
Copy link
Collaborator

@sumeerbhola sumeerbhola left a comment

Choose a reason for hiding this comment

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

thanks

@annrpom
Copy link
Contributor Author

annrpom commented Nov 6, 2025

np; TFTRs! ('-')7

bors r+

craig bot pushed a commit that referenced this pull request Nov 6, 2025
156970: ui: add store disk write bytes to storage dashboard r=annrpom a=annrpom

This patch adds a metric for store disk write bytes in the Storage dashboard.

Epic: none
Informs: https://github.com/cockroachlabs/support/issues/3470

Release note: None

<img width="720" height="308" alt="image" src="https://github.com/user-attachments/assets/94629b53-2dc7-4a42-8b68-85ddbd93d006" />
<img width="968" height="411" alt="image" src="https://github.com/user-attachments/assets/e7136191-8df7-4dcf-8150-815661395af1" />

<img width="968" height="407" alt="image" src="https://github.com/user-attachments/assets/3add06df-0f76-48c5-b467-aca2f9742869" />
<img width="975" height="414" alt="image" src="https://github.com/user-attachments/assets/0b6b5812-be09-4def-83e4-96ce57dd36de" />


Co-authored-by: Annie Pompa <annie@cockroachlabs.com>
@craig
Copy link
Contributor

craig bot commented Nov 6, 2025

Build failed:

@annrpom
Copy link
Contributor Author

annrpom commented Nov 7, 2025

bors retry

@craig
Copy link
Contributor

craig bot commented Nov 7, 2025

@craig craig bot merged commit 5669040 into cockroachdb:master Nov 7, 2025
24 checks passed
@annrpom annrpom deleted the storage-dashboard-bytes branch November 7, 2025 20:37
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.

4 participants