Skip to content

grafana-tikv-dashboard.md: Resolved TS row and panel names are out of sync with TiKV #23921

Description

@sary8

Error Report

Please answer the following questions before submitting your issue. Thanks!

  1. What is the URL/path of the document related to this issue?
  1. How would you like to improve it?

The Resolved-TS section uses row and panel names that do not match the current TiKV dashboard source. The name in the docs is on the left, and the dashboard title is on the right:

  • Resolved-TSResolved TS (row title)
  • Resolved-TS worker CPUResolved TS Worker CPU
  • Advance-TS worker CPUAdvance ts Worker CPU
  • Scan lock worker CPUScan lock Worker CPU
  • Max gap of safe-tsMax gap of follower safe-ts
  • Min Safe TS RegionMin Safe TS Follower Region
  • Check Leader DurationCheck leader duration
  • Max gap of resolved-ts in Region leadersMax gap of resolved-ts in region leaders

Source: tikv_details.dashboard.py#L8999-L9255

All eight names should be updated. For six of them, only the label changes. The two safe-ts entries also need their descriptions updated because the underlying panels now measure follower-specific values:

  • Max gap of follower safe-ts queries tikv_resolved_ts_min_follower_safe_ts_gap_millis and is described in TiKV as the gap between the current time and the minimum non-zero safe-ts for followers. The docs currently describe the safe-ts of all active Regions.
  • Min Safe TS Follower Region queries tikv_resolved_ts_min_follower_safe_ts_region and reports the follower Region with the minimum safe-ts. The docs currently describe the Region with the minimum safe-ts without the follower scope.

The remaining existing labels in this docs section already match the dashboard: Max gap of resolved-ts, Min Resolved TS Region, Min Leader Resolved TS Region, and Lock heap size.

Renaming the section heading from Resolved-TS to Resolved TS does not change the #resolved-ts anchor because both headings produce the same slug.

Related to #23818, which covers the same type of Grafana naming mismatch in other files. grafana-tikv-dashboard.md is not one of the files listed there.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

contributionThis PR is from a community contributor.first-time-contributorIndicates that the PR was contributed by an external member and is a first-time contributor.

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions