Skip to content

Commit ecd5dd4

Browse files
authored
chore: remove unnecessary for delete errors (#1770)
Signed-off-by: helen <[email protected]>
1 parent ed8f88e commit ecd5dd4

File tree

1 file changed

+2
-6
lines changed

1 file changed

+2
-6
lines changed

pkg/controllers/provisioning/scheduling/scheduler.go

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -217,6 +217,7 @@ func (s *Scheduler) Solve(ctx context.Context, pods []*corev1.Pod) Results {
217217

218218
// Schedule to existing nodes or create a new node
219219
if errors[pod] = s.add(ctx, pod); errors[pod] == nil {
220+
delete(errors, pod)
220221
continue
221222
}
222223

@@ -233,12 +234,7 @@ func (s *Scheduler) Solve(ctx context.Context, pods []*corev1.Pod) Results {
233234
for _, m := range s.newNodeClaims {
234235
m.FinalizeScheduling()
235236
}
236-
// clear any nil errors, so we can know that len(PodErrors) == 0 => all pods scheduled
237-
for k, v := range errors {
238-
if v == nil {
239-
delete(errors, k)
240-
}
241-
}
237+
242238
return Results{
243239
NewNodeClaims: s.newNodeClaims,
244240
ExistingNodes: s.existingNodes,

0 commit comments

Comments
 (0)