Skip to content

ui: add charts on gc assist metric#119506

Merged
craig[bot] merged 1 commit into
cockroachdb:masterfrom
lyang24:gc_assist_ui
Mar 4, 2024
Merged

ui: add charts on gc assist metric#119506
craig[bot] merged 1 commit into
cockroachdb:masterfrom
lyang24:gc_assist_ui

Conversation

@lyang24

@lyang24 lyang24 commented Feb 22, 2024

Copy link
Copy Markdown
Contributor

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

@blathers-crl

blathers-crl Bot commented Feb 22, 2024

Copy link
Copy Markdown

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
@blathers-crl blathers-crl Bot requested a review from nvb February 22, 2024 05:12
@cockroach-teamcity

Copy link
Copy Markdown
Member

This change is Reviewable

@lyang24 lyang24 left a comment

Copy link
Copy Markdown
Contributor Author

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
@blathers-crl

blathers-crl Bot commented Feb 22, 2024

Copy link
Copy Markdown

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

@nvb nvb left a comment

Copy link
Copy Markdown
Contributor

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

lyang24 commented Mar 4, 2024

Copy link
Copy Markdown
Contributor Author

bors r+

@craig

craig Bot commented Mar 4, 2024

Copy link
Copy Markdown
Contributor

Build succeeded:

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