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

ui: add charts on gc assist metric #119506

Merged
merged 1 commit into from
Mar 4, 2024
Merged

Conversation

lyang24
Copy link
Contributor

@lyang24 lyang24 commented Feb 22, 2024

This commit adds a line chart contains gc assist duration on the runtime page. The goal is to present the estimated time user go routines spend on assisting gc tasks.

Informs: #118875

Release note: None

gc assist on crdb single node with kv workload gcttl=5s for database in (kv, system)
image

Copy link

blathers-crl bot commented Feb 22, 2024

Thank you for contributing to CockroachDB. Please ensure you have followed the guidelines for creating a PR.

My owl senses detect your PR is good for review. Please keep an eye out for any test failures in CI.

I have added a few people who may be able to assist in reviewing:

🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf.

@blathers-crl blathers-crl bot added O-community Originated from the community X-blathers-triaged blathers was able to find an owner labels Feb 22, 2024
@cockroach-teamcity
Copy link
Member

This change is Reviewable

Copy link
Contributor Author

@lyang24 lyang24 left a comment

Choose a reason for hiding this comment

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

Hi, i found a couple of examples monitoring metric examples are not up to date (as they still have network under hardware session and previous packet error and drop metrics missing).
I generated the grafana runtime.json file (based on monitoring/README.md) on my local machine it seems like a lot of a lot of fields were added to the previous code blocks possible due to the grafana version. I wonder if we should update them as a separate effort with another ticket to track it.

pkg/cmd/roachprod/grafana
monitoring/splunk-dashboard
monitoring/grafana-dashboards

Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained (waiting on @nvanbenschoten)

This commit adds a line chart contains gc assist duration on the runtime page.
The goal is to present the estimated time user go routines spend on assisting
gc tasks.

Informs: cockroachdb#118875

Release note: None
Copy link

blathers-crl bot commented Feb 22, 2024

Thank you for updating your pull request.

My owl senses detect your PR is good for review. Please keep an eye out for any test failures in CI.

🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf.

@lyang24 lyang24 marked this pull request as ready for review February 22, 2024 06:06
@lyang24 lyang24 requested a review from a team February 22, 2024 06:06
Copy link
Member

@nvanbenschoten nvanbenschoten left a comment

Choose a reason for hiding this comment

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

:lgtm:

I wonder if we should update them as a separate effort with another ticket to track it.

I think this makes sense. @dhartunian what is our current strategy for keeping all of this different metrics dashboards in sync?

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! 1 of 0 LGTMs obtained (waiting on @lyang24)

@lyang24
Copy link
Contributor Author

lyang24 commented Mar 4, 2024

bors r+

@craig
Copy link
Contributor

craig bot commented Mar 4, 2024

Build succeeded:

@craig craig bot merged commit 1cab3ba into cockroachdb:master Mar 4, 2024
20 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
O-community Originated from the community X-blathers-triaged blathers was able to find an owner
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants