Skip to content
This repository has been archived by the owner on Jun 24, 2021. It is now read-only.

scripts: add store back to metrics #737

Merged
merged 1 commit into from
Apr 23, 2019
Merged

Conversation

rleungx
Copy link
Member

@rleungx rleungx commented Apr 22, 2019

Related to tikv/pd#1506.

Signed-off-by: Ryan Leung <rleungx@gmail.com>
@rleungx rleungx requested a review from nolouch April 22, 2019 03:43
@CLAassistant
Copy link

CLAassistant commented Apr 22, 2019

CLA assistant check
All committers have signed the CLA.

@@ -1713,10 +1713,10 @@
"steppedLine": false,
"targets": [
{
"expr": "sum(pd_scheduler_store_status{instance=\"$instance\", namespace=~\"$namespace\", type=\"store_available\"}) by (address) / sum(pd_scheduler_store_status{instance=\"$instance\", namespace=~\"$namespace\", type=\"store_capacity\"}) by (address)",
"expr": "sum(pd_scheduler_store_status{instance=\"$instance\", namespace=~\"$namespace\", type=\"store_available\"}) by (address, store) / sum(pd_scheduler_store_status{instance=\"$instance\", namespace=~\"$namespace\", type=\"store_capacity\"}) by (address, store)",
Copy link
Member

@nolouch nolouch Apr 23, 2019

Choose a reason for hiding this comment

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

Is only by store already enough?

@nolouch nolouch merged commit e767308 into pingcap:master Apr 23, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants