Skip to content

Commit 71e7f9e

Browse files
authored
Merge pull request kubernetes#131425 from carlory/follow-up-128810
Remove deprecated scheduler cache metrics
2 parents 233ebd6 + 24257f2 commit 71e7f9e

File tree

2 files changed

+2
-23
lines changed

2 files changed

+2
-23
lines changed

pkg/scheduler/backend/cache/cache.go

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -757,12 +757,4 @@ func (cache *cacheImpl) updateMetrics() {
757757
metrics.CacheSize.WithLabelValues("assumed_pods").Set(float64(len(cache.assumedPods)))
758758
metrics.CacheSize.WithLabelValues("pods").Set(float64(len(cache.podStates)))
759759
metrics.CacheSize.WithLabelValues("nodes").Set(float64(len(cache.nodes)))
760-
761-
// we intentionally keep them with the deprecation and will remove at v1.34.
762-
//nolint:staticcheck
763-
metrics.SchedulerCacheSize.WithLabelValues("assumed_pods").Set(float64(len(cache.assumedPods)))
764-
//nolint:staticcheck
765-
metrics.SchedulerCacheSize.WithLabelValues("pods").Set(float64(len(cache.podStates)))
766-
//nolint:staticcheck
767-
metrics.SchedulerCacheSize.WithLabelValues("nodes").Set(float64(len(cache.nodes)))
768760
}

pkg/scheduler/metrics/metrics.go

Lines changed: 2 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -107,11 +107,8 @@ var (
107107
FrameworkExtensionPointDuration *metrics.HistogramVec
108108
PluginExecutionDuration *metrics.HistogramVec
109109

110-
PermitWaitDuration *metrics.HistogramVec
111-
CacheSize *metrics.GaugeVec
112-
// Deprecated: SchedulerCacheSize is deprecated,
113-
// and will be removed at v1.34. Please use CacheSize instead.
114-
SchedulerCacheSize *metrics.GaugeVec
110+
PermitWaitDuration *metrics.HistogramVec
111+
CacheSize *metrics.GaugeVec
115112
unschedulableReasons *metrics.GaugeVec
116113
PluginEvaluationTotal *metrics.CounterVec
117114

@@ -294,15 +291,6 @@ func InitMetrics() {
294291
},
295292
[]string{"result"})
296293

297-
SchedulerCacheSize = metrics.NewGaugeVec(
298-
&metrics.GaugeOpts{
299-
Subsystem: SchedulerSubsystem,
300-
Name: "scheduler_cache_size",
301-
Help: "Number of nodes, pods, and assumed (bound) pods in the scheduler cache.",
302-
StabilityLevel: metrics.ALPHA,
303-
DeprecatedVersion: "1.33.0",
304-
}, []string{"type"})
305-
306294
CacheSize = metrics.NewGaugeVec(
307295
&metrics.GaugeOpts{
308296
Subsystem: SchedulerSubsystem,
@@ -362,7 +350,6 @@ func InitMetrics() {
362350
Goroutines,
363351
PermitWaitDuration,
364352
CacheSize,
365-
SchedulerCacheSize,
366353
unschedulableReasons,
367354
PluginEvaluationTotal,
368355
}

0 commit comments

Comments
 (0)