Skip to content

Commit 9767696

Browse files
author
sunxiaofei03
committed
deprecate scheduling_algorithm_predicate/priority_evaluation_seconds
1 parent 5e31799 commit 9767696

File tree

2 files changed

+19
-17
lines changed

2 files changed

+19
-17
lines changed

pkg/scheduler/core/generic_scheduler.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -218,15 +218,15 @@ func (g *genericScheduler) Schedule(ctx context.Context, state *framework.CycleS
218218
}
219219
}
220220
trace.Step("Running postfilter plugins done")
221-
metrics.SchedulingAlgorithmPredicateEvaluationDuration.Observe(metrics.SinceInSeconds(startPredicateEvalTime))
221+
metrics.DeprecatedSchedulingAlgorithmPredicateEvaluationSecondsDuration.Observe(metrics.SinceInSeconds(startPredicateEvalTime))
222222
metrics.DeprecatedSchedulingAlgorithmPredicateEvaluationDuration.Observe(metrics.SinceInMicroseconds(startPredicateEvalTime))
223223
metrics.SchedulingLatency.WithLabelValues(metrics.PredicateEvaluation).Observe(metrics.SinceInSeconds(startPredicateEvalTime))
224224
metrics.DeprecatedSchedulingLatency.WithLabelValues(metrics.PredicateEvaluation).Observe(metrics.SinceInSeconds(startPredicateEvalTime))
225225

226226
startPriorityEvalTime := time.Now()
227227
// When only one node after predicate, just use it.
228228
if len(filteredNodes) == 1 {
229-
metrics.SchedulingAlgorithmPriorityEvaluationDuration.Observe(metrics.SinceInSeconds(startPriorityEvalTime))
229+
metrics.DeprecatedSchedulingAlgorithmPriorityEvaluationSecondsDuration.Observe(metrics.SinceInSeconds(startPriorityEvalTime))
230230
metrics.DeprecatedSchedulingAlgorithmPriorityEvaluationDuration.Observe(metrics.SinceInMicroseconds(startPriorityEvalTime))
231231
return ScheduleResult{
232232
SuggestedHost: filteredNodes[0].Name,
@@ -241,7 +241,7 @@ func (g *genericScheduler) Schedule(ctx context.Context, state *framework.CycleS
241241
return result, err
242242
}
243243

244-
metrics.SchedulingAlgorithmPriorityEvaluationDuration.Observe(metrics.SinceInSeconds(startPriorityEvalTime))
244+
metrics.DeprecatedSchedulingAlgorithmPriorityEvaluationSecondsDuration.Observe(metrics.SinceInSeconds(startPriorityEvalTime))
245245
metrics.DeprecatedSchedulingAlgorithmPriorityEvaluationDuration.Observe(metrics.SinceInMicroseconds(startPriorityEvalTime))
246246
metrics.SchedulingLatency.WithLabelValues(metrics.PriorityEvaluation).Observe(metrics.SinceInSeconds(startPriorityEvalTime))
247247
metrics.DeprecatedSchedulingLatency.WithLabelValues(metrics.PriorityEvaluation).Observe(metrics.SinceInSeconds(startPriorityEvalTime))

pkg/scheduler/metrics/metrics.go

Lines changed: 16 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -126,13 +126,14 @@ var (
126126
DeprecatedVersion: "1.14.0",
127127
},
128128
)
129-
SchedulingAlgorithmPredicateEvaluationDuration = metrics.NewHistogram(
129+
DeprecatedSchedulingAlgorithmPredicateEvaluationSecondsDuration = metrics.NewHistogram(
130130
&metrics.HistogramOpts{
131-
Subsystem: SchedulerSubsystem,
132-
Name: "scheduling_algorithm_predicate_evaluation_seconds",
133-
Help: "Scheduling algorithm predicate evaluation duration in seconds",
134-
Buckets: metrics.ExponentialBuckets(0.001, 2, 15),
135-
StabilityLevel: metrics.ALPHA,
131+
Subsystem: SchedulerSubsystem,
132+
Name: "scheduling_algorithm_predicate_evaluation_seconds",
133+
Help: "Scheduling algorithm predicate evaluation duration in seconds",
134+
Buckets: metrics.ExponentialBuckets(0.001, 2, 15),
135+
StabilityLevel: metrics.ALPHA,
136+
DeprecatedVersion: "1.18.0",
136137
},
137138
)
138139
DeprecatedSchedulingAlgorithmPredicateEvaluationDuration = metrics.NewHistogram(
@@ -145,13 +146,14 @@ var (
145146
DeprecatedVersion: "1.14.0",
146147
},
147148
)
148-
SchedulingAlgorithmPriorityEvaluationDuration = metrics.NewHistogram(
149+
DeprecatedSchedulingAlgorithmPriorityEvaluationSecondsDuration = metrics.NewHistogram(
149150
&metrics.HistogramOpts{
150-
Subsystem: SchedulerSubsystem,
151-
Name: "scheduling_algorithm_priority_evaluation_seconds",
152-
Help: "Scheduling algorithm priority evaluation duration in seconds",
153-
Buckets: metrics.ExponentialBuckets(0.001, 2, 15),
154-
StabilityLevel: metrics.ALPHA,
151+
Subsystem: SchedulerSubsystem,
152+
Name: "scheduling_algorithm_priority_evaluation_seconds",
153+
Help: "Scheduling algorithm priority evaluation duration in seconds",
154+
Buckets: metrics.ExponentialBuckets(0.001, 2, 15),
155+
StabilityLevel: metrics.ALPHA,
156+
DeprecatedVersion: "1.18.0",
155157
},
156158
)
157159
DeprecatedSchedulingAlgorithmPriorityEvaluationDuration = metrics.NewHistogram(
@@ -311,9 +313,9 @@ var (
311313
DeprecatedSchedulingAlgorithmLatency,
312314
BindingLatency,
313315
DeprecatedBindingLatency,
314-
SchedulingAlgorithmPredicateEvaluationDuration,
316+
DeprecatedSchedulingAlgorithmPredicateEvaluationSecondsDuration,
315317
DeprecatedSchedulingAlgorithmPredicateEvaluationDuration,
316-
SchedulingAlgorithmPriorityEvaluationDuration,
318+
DeprecatedSchedulingAlgorithmPriorityEvaluationSecondsDuration,
317319
DeprecatedSchedulingAlgorithmPriorityEvaluationDuration,
318320
SchedulingAlgorithmPreemptionEvaluationDuration,
319321
DeprecatedSchedulingAlgorithmPreemptionEvaluationDuration,

0 commit comments

Comments
 (0)