mirror of https://github.com/k3s-io/k3s
Merge pull request #45091 from k82cn/update_weight_check
Automatic merge from submit-queue Align Extender's validation with prioritizers. **What this PR does / why we need it**: Align Extender's validation with prioritizers. **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes # N/A **Release note**: ```release-note-none ```pull/6/head
commit
05fafe1525
|
@ -35,8 +35,8 @@ func ValidatePolicy(policy schedulerapi.Policy) error {
|
||||||
}
|
}
|
||||||
|
|
||||||
for _, extender := range policy.ExtenderConfigs {
|
for _, extender := range policy.ExtenderConfigs {
|
||||||
if extender.Weight < 0 {
|
if extender.Weight <= 0 {
|
||||||
validationErrors = append(validationErrors, fmt.Errorf("Priority for extender %s should have a non negative weight applied to it", extender.URLPrefix))
|
validationErrors = append(validationErrors, fmt.Errorf("Priority for extender %s should have a positive weight applied to it", extender.URLPrefix))
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
return utilerrors.NewAggregate(validationErrors)
|
return utilerrors.NewAggregate(validationErrors)
|
||||||
|
|
Loading…
Reference in New Issue