DevFW-CICD/ingress-nginx-helm
14
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request #2844 from aledbf/fix-latency

Browse source 
Do not allow invalid latency values in metrics
This commit is contained in:
k8s-ci-robot 2018-07-24 08:28:55 -07:00 committed by GitHub
commit bdb5ddc473
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 7 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
internal/ingress/metric/collectors/socket.go
View file

@ -236,12 +236,14 @@ func (sc *SocketCollector) handleMessage(msg []byte) {
requestsMetric.Inc() requestsMetric.Inc()
} }
if stats.Latency != -1 {
latencyMetric, err := sc.upstreamLatency.GetMetricWith(latencyLabels) latencyMetric, err := sc.upstreamLatency.GetMetricWith(latencyLabels)
if err != nil { if err != nil {
glog.Errorf("Error fetching latency metric: %v", err) glog.Errorf("Error fetching latency metric: %v", err)
} else { } else {
latencyMetric.Observe(stats.Latency) latencyMetric.Observe(stats.Latency)
} }
}
if stats.RequestTime != -1 { if stats.RequestTime != -1 {
requestTimeMetric, err := sc.requestTime.GetMetricWith(requestLabels) requestTimeMetric, err := sc.requestTime.GetMetricWith(requestLabels)