Skip to content

Commit ae6b552

Browse files
committed
DRA scheduler: rename variable
"Allocated devices" are the ones which can be observed from the informer. "All allocated devices" also includes those which are in flight and haven't been written back to the apiserver.
1 parent 0130ebb commit ae6b552

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

pkg/scheduler/framework/plugins/dynamicresources/dynamicresources.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -545,12 +545,12 @@ func (pl *DynamicResources) PreFilter(ctx context.Context, state *framework.Cycl
545545
// Claims are treated as "allocated" if they are in the assume cache
546546
// or currently their allocation is in-flight. This does not change
547547
// during filtering, so we can determine that once.
548-
allocatedDevices := pl.listAllAllocatedDevices(logger)
548+
allAllocatedDevices := pl.listAllAllocatedDevices(logger)
549549
slices, err := pl.sliceLister.List(labels.Everything())
550550
if err != nil {
551551
return nil, statusError(logger, err)
552552
}
553-
allocator, err := structured.NewAllocator(ctx, pl.enableAdminAccess, allocateClaims, allocatedDevices, pl.classLister, slices, pl.celCache)
553+
allocator, err := structured.NewAllocator(ctx, pl.enableAdminAccess, allocateClaims, allAllocatedDevices, pl.classLister, slices, pl.celCache)
554554
if err != nil {
555555
return nil, statusError(logger, err)
556556
}

0 commit comments

Comments
 (0)