clean deprecated apiserver request metrics
This commit is contained in:
		| @@ -74,16 +74,6 @@ var ( | |||||||
| 		// should be all lowercase and separated by underscores. | 		// should be all lowercase and separated by underscores. | ||||||
| 		[]string{"verb", "dry_run", "group", "version", "resource", "subresource", "scope", "component", "client", "contentType", "code"}, | 		[]string{"verb", "dry_run", "group", "version", "resource", "subresource", "scope", "component", "client", "contentType", "code"}, | ||||||
| 	) | 	) | ||||||
| 	// TODO(logicalhan): remove this deprecated metrics after fixing the dependencies |  | ||||||
| 	deprecatedRequestCounter = compbasemetrics.NewCounterVec( |  | ||||||
| 		&compbasemetrics.CounterOpts{ |  | ||||||
| 			Name:              "apiserver_request_count", |  | ||||||
| 			Help:              "Counter of apiserver requests broken out for each verb, group, version, resource, scope, component, client, and HTTP response contentType and code.", |  | ||||||
| 			StabilityLevel:    compbasemetrics.ALPHA, |  | ||||||
| 			DeprecatedVersion: "1.14.0", |  | ||||||
| 		}, |  | ||||||
| 		[]string{"verb", "group", "version", "resource", "subresource", "scope", "component", "client", "contentType", "code"}, |  | ||||||
| 	) |  | ||||||
| 	longRunningRequestGauge = compbasemetrics.NewGaugeVec( | 	longRunningRequestGauge = compbasemetrics.NewGaugeVec( | ||||||
| 		&compbasemetrics.GaugeOpts{ | 		&compbasemetrics.GaugeOpts{ | ||||||
| 			Name:           "apiserver_longrunning_gauge", | 			Name:           "apiserver_longrunning_gauge", | ||||||
| @@ -105,19 +95,6 @@ var ( | |||||||
| 		}, | 		}, | ||||||
| 		[]string{"verb", "dry_run", "group", "version", "resource", "subresource", "scope", "component"}, | 		[]string{"verb", "dry_run", "group", "version", "resource", "subresource", "scope", "component"}, | ||||||
| 	) | 	) | ||||||
| 	// TODO(wojtek-t): remove this deprecated metrics after fixing the dependencies |  | ||||||
| 	deprecatedRequestLatenciesSummary = compbasemetrics.NewSummaryVec( |  | ||||||
| 		&compbasemetrics.SummaryOpts{ |  | ||||||
| 			Name: "apiserver_request_latencies_summary", |  | ||||||
| 			Help: "Response latency summary in microseconds for each verb, group, version, resource, subresource, scope and component.", |  | ||||||
| 			// Make the sliding window of 5h. |  | ||||||
| 			// TODO: The value for this should be based on our SLI definition (medium term). |  | ||||||
| 			MaxAge:            5 * time.Hour, |  | ||||||
| 			StabilityLevel:    compbasemetrics.ALPHA, |  | ||||||
| 			DeprecatedVersion: "1.14.0", |  | ||||||
| 		}, |  | ||||||
| 		[]string{"verb", "group", "version", "resource", "subresource", "scope", "component"}, |  | ||||||
| 	) |  | ||||||
| 	responseSizes = compbasemetrics.NewHistogramVec( | 	responseSizes = compbasemetrics.NewHistogramVec( | ||||||
| 		&compbasemetrics.HistogramOpts{ | 		&compbasemetrics.HistogramOpts{ | ||||||
| 			Name: "apiserver_response_sizes", | 			Name: "apiserver_response_sizes", | ||||||
| @@ -186,10 +163,8 @@ var ( | |||||||
|  |  | ||||||
| 	metrics = []resettableCollector{ | 	metrics = []resettableCollector{ | ||||||
| 		requestCounter, | 		requestCounter, | ||||||
| 		deprecatedRequestCounter, |  | ||||||
| 		longRunningRequestGauge, | 		longRunningRequestGauge, | ||||||
| 		requestLatencies, | 		requestLatencies, | ||||||
| 		deprecatedRequestLatenciesSummary, |  | ||||||
| 		responseSizes, | 		responseSizes, | ||||||
| 		DroppedRequests, | 		DroppedRequests, | ||||||
| 		RegisteredWatchers, | 		RegisteredWatchers, | ||||||
| @@ -282,12 +257,9 @@ func MonitorRequest(req *http.Request, verb, group, version, resource, subresour | |||||||
| 	reportedVerb := cleanVerb(verb, req) | 	reportedVerb := cleanVerb(verb, req) | ||||||
| 	dryRun := cleanDryRun(req.URL) | 	dryRun := cleanDryRun(req.URL) | ||||||
| 	client := cleanUserAgent(utilnet.GetHTTPClient(req)) | 	client := cleanUserAgent(utilnet.GetHTTPClient(req)) | ||||||
| 	elapsedMicroseconds := float64(elapsed / time.Microsecond) |  | ||||||
| 	elapsedSeconds := elapsed.Seconds() | 	elapsedSeconds := elapsed.Seconds() | ||||||
| 	requestCounter.WithLabelValues(reportedVerb, dryRun, group, version, resource, subresource, scope, component, client, contentType, codeToString(httpCode)).Inc() | 	requestCounter.WithLabelValues(reportedVerb, dryRun, group, version, resource, subresource, scope, component, client, contentType, codeToString(httpCode)).Inc() | ||||||
| 	deprecatedRequestCounter.WithLabelValues(reportedVerb, group, version, resource, subresource, scope, component, client, contentType, codeToString(httpCode)).Inc() |  | ||||||
| 	requestLatencies.WithLabelValues(reportedVerb, dryRun, group, version, resource, subresource, scope, component).Observe(elapsedSeconds) | 	requestLatencies.WithLabelValues(reportedVerb, dryRun, group, version, resource, subresource, scope, component).Observe(elapsedSeconds) | ||||||
| 	deprecatedRequestLatenciesSummary.WithLabelValues(reportedVerb, group, version, resource, subresource, scope, component).Observe(elapsedMicroseconds) |  | ||||||
| 	// We are only interested in response sizes of read requests. | 	// We are only interested in response sizes of read requests. | ||||||
| 	if verb == "GET" || verb == "LIST" { | 	if verb == "GET" || verb == "LIST" { | ||||||
| 		responseSizes.WithLabelValues(reportedVerb, group, version, resource, subresource, scope, component).Observe(float64(respSize)) | 		responseSizes.WithLabelValues(reportedVerb, group, version, resource, subresource, scope, component).Observe(float64(respSize)) | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user
	 danielqsj
					danielqsj