Skip to content

Commit

Permalink
Review remark
Browse files Browse the repository at this point in the history
  • Loading branch information
yaroslava-serdiuk committed Sep 11, 2024
1 parent dd9822a commit e0fb624
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion cluster-autoscaler/processors/provreq/injector.go
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,6 @@ func (p *ProvisioningRequestPodsInjector) GetPodsFromNextRequest(
}
for _, pr := range provReqs {
if !isSupportedClass(pr) {
klog.Warningf("Provisioning Class %s is not supported for ProvReq %s/%s", pr.Spec.ProvisioningClassName, pr.Namespace, pr.Name)
continue
}
conditions := pr.Status.Conditions
Expand Down Expand Up @@ -135,6 +134,9 @@ func (p *ProvisioningRequestPodsInjector) Process(
podsFromProvReq, err := p.GetPodsFromNextRequest(
func(pr *provreqwrapper.ProvisioningRequest) bool {
_, found := provisioningrequest.SupportedProvisioningClasses[pr.Spec.ProvisioningClassName]
if !found {
klog.Warningf("Provisioning Class %s is not supported for ProvReq %s/%s", pr.Spec.ProvisioningClassName, pr.Namespace, pr.Name)
}
return found
})

Expand Down

0 comments on commit e0fb624

Please sign in to comment.