diff --git a/pkg/controllers/disruption/multinodeconsolidation.go b/pkg/controllers/disruption/multinodeconsolidation.go index e033aa176b..2e2039c1a0 100644 --- a/pkg/controllers/disruption/multinodeconsolidation.go +++ b/pkg/controllers/disruption/multinodeconsolidation.go @@ -209,7 +209,10 @@ func filterOutSameType(newNodeClaim *scheduling.NodeClaim, consolidate []*Candid } // swallow the error since we don't allow min values to impact reschedulability in multi node claim newNodeClaim, err := newNodeClaim.RemoveInstanceTypeOptionsByPriceAndMinValues(newNodeClaim.Requirements, maxPrice) - return newNodeClaim.InstanceTypeOptions, err + if err != nil { + return nil, err + } + return newNodeClaim.InstanceTypeOptions, nil } func (m *MultiNodeConsolidation) Reason() v1.DisruptionReason {