Skip to content

Add remaining durability levels to durable latencies graph#140

Open
AlphaDaze wants to merge 2 commits intocouchbaselabs:masterfrom
AlphaDaze:durable-latencies
Open

Add remaining durability levels to durable latencies graph#140
AlphaDaze wants to merge 2 commits intocouchbaselabs:masterfrom
AlphaDaze:durable-latencies

Conversation

@AlphaDaze
Copy link
Copy Markdown
Contributor

Add majority and majority-and-persist-on-master to Percentile Sync Write latencies dashboard.

Currently only persist_to_majority is plotted.

Copilot AI review requested due to automatic review settings April 23, 2026 15:37
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the KV cluster Grafana dashboard to plot additional sync-write durability levels in the “Percentile Sync Write latencies” panel, so operators can compare latency across durability modes instead of only seeing persist_to_majority.

Changes:

  • Add new Prometheus queries for majority and majority_and_persist_on_master durability levels (p50 and p99).
  • Update existing legend labels for persist_to_majority to reflect the durability level.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread dashboards/kv-cluster.json
Comment thread dashboards/kv-cluster.json
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread dashboards/kv-cluster.json
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants