Skip to content

Commit

Permalink
backend: add prometheus metric for large snapshot duration.
Browse files Browse the repository at this point in the history
FIXES #7878
  • Loading branch information
fanminshi committed May 6, 2017
1 parent db6f45e commit 230106d
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 0 deletions.
1 change: 1 addition & 0 deletions mvcc/backend/backend.go
Original file line number Diff line number Diff line change
Expand Up @@ -176,6 +176,7 @@ func (b *backend) Snapshot() Snapshot {
case <-ticker.C:
plog.Warningf("snapshotting is taking more than %v seconds to finish [started at %v]", time.Since(start).Seconds(), start)
case <-stopc:
snapshotDurations.Observe(time.Since(start).Seconds())
return
}
}
Expand Down
10 changes: 10 additions & 0 deletions mvcc/backend/metrics.go
Original file line number Diff line number Diff line change
Expand Up @@ -24,8 +24,18 @@ var (
Help: "The latency distributions of commit called by backend.",
Buckets: prometheus.ExponentialBuckets(0.001, 2, 14),
})

snapshotDurations = prometheus.NewHistogram(prometheus.HistogramOpts{
Namespace: "etcd",
Subsystem: "disk",
Name: "backend_snapshot_duration_seconds",
Help: "The latency distribution of backend snapshots.",
// 10 ms -> 655 seconds
Buckets: prometheus.ExponentialBuckets(.01, 2, 17),
})
)

func init() {
prometheus.MustRegister(commitDurations)
prometheus.MustRegister(snapshotDurations)
}

0 comments on commit 230106d

Please sign in to comment.