Skip to content

Commit db78712

Browse files
authored
Merge pull request #901 from yehlemias/fix-reject-rnb-pod-with-sidecar
fix reject rnb pod with sidecar
2 parents adacc8c + 84dec67 commit db78712

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

pkg/agent/qrm-plugins/cpu/dynamicpolicy/policy_allocation_handlers.go

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -197,13 +197,17 @@ func (p *DynamicPolicy) sharedCoresWithoutNUMABindingAllocationHandler(_ context
197197
return resp, nil
198198
}
199199

200-
func (p *DynamicPolicy) reclaimedCoresAllocationHandler(_ context.Context,
200+
func (p *DynamicPolicy) reclaimedCoresAllocationHandler(ctx context.Context,
201201
req *pluginapi.ResourceRequest, persistCheckpoint bool,
202202
) (*pluginapi.ResourceAllocationResponse, error) {
203203
if req == nil {
204204
return nil, fmt.Errorf("reclaimedCoresAllocationHandler got nil request")
205205
}
206206

207+
if req.ContainerType == pluginapi.ContainerType_SIDECAR {
208+
return p.allocationSidecarHandler(ctx, req, apiconsts.PodAnnotationQoSLevelReclaimedCores, persistCheckpoint)
209+
}
210+
207211
if util.PodInplaceUpdateResizing(req) {
208212
return nil, fmt.Errorf("not support inplace update resize for reclaimed cores")
209213
}

0 commit comments

Comments
 (0)