github/kubernetes
4
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix minor bugs in setting API call metrics with subresource

Browse Source
This commit is contained in:
Shyam Jeedigunta
2017-05-29 15:04:52 +02:00
parent 25aa600360
commit e897b21506
2 changed files with 6 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
test/e2e/framework/metrics_util.go
View File

@@ -224,7 +224,7 @@ func (a *APIResponsiveness) Less(i, j int) bool {
// Only 0.5, 0.9 and 0.99 quantiles are supported. // Only 0.5, 0.9 and 0.99 quantiles are supported.
func (a *APIResponsiveness) addMetricRequestLatency(resource, subresource, verb string, quantile float64, latency time.Duration) { func (a *APIResponsiveness) addMetricRequestLatency(resource, subresource, verb string, quantile float64, latency time.Duration) {
for i, apicall := range a.APICalls { for i, apicall := range a.APICalls {
if apicall.Resource == resource && apicall.Verb == verb { if apicall.Resource == resource && apicall.Subresource == subresource && apicall.Verb == verb {
a.APICalls[i] = setQuantileAPICall(apicall, quantile, latency) a.APICalls[i] = setQuantileAPICall(apicall, quantile, latency)
return return
} }
@@ -255,7 +255,7 @@ func setQuantile(metric *LatencyMetric, quantile float64, latency time.Duration)
// Add request count to the APICall metric entry (creating one if necessary). // Add request count to the APICall metric entry (creating one if necessary).
func (a *APIResponsiveness) addMetricRequestCount(resource, subresource, verb string, count int) { func (a *APIResponsiveness) addMetricRequestCount(resource, subresource, verb string, count int) {
for i, apicall := range a.APICalls { for i, apicall := range a.APICalls {
if apicall.Resource == resource && apicall.Verb == verb { if apicall.Resource == resource && apicall.Subresource == subresource && apicall.Verb == verb {
a.APICalls[i].Count += count a.APICalls[i].Count += count
return return
} }

1
test/e2e/framework/perf_util.go
View File

@@ -43,6 +43,7 @@ func ApiCallToPerfData(apicalls *APIResponsiveness) *perftype.PerfData {
Labels: map[string]string{ Labels: map[string]string{
"Verb": apicall.Verb, "Verb": apicall.Verb,
"Resource": apicall.Resource, "Resource": apicall.Resource,
"Subresource": apicall.Subresource,
"Count": fmt.Sprintf("%v", apicall.Count), "Count": fmt.Sprintf("%v", apicall.Count),
}, },
} }