Skip to content

Commit

Permalink
includeSoftConstraints not being respected for TopologySpreadConstraint
Browse files Browse the repository at this point in the history
Issue #960

Signed-off-by: Vlastimil Holer <vh@fortrabbit.com>
  • Loading branch information
vholer committed Sep 26, 2022
1 parent 1d79f87 commit 7349b39
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pkg/descheduler/strategy_migration.go
Original file line number Diff line number Diff line change
Expand Up @@ -212,7 +212,7 @@ var pluginsMap = map[string]func(ctx context.Context, nodes []*v1.Node, params *
args := &componentconfig.RemovePodsViolatingTopologySpreadConstraintArgs{
Namespaces: params.Namespaces,
LabelSelector: params.LabelSelector,
IncludeSoftConstraints: params.IncludePreferNoSchedule,
IncludeSoftConstraints: params.IncludeSoftConstraints,
}
if err := validation.ValidateRemovePodsViolatingTopologySpreadConstraintArgs(args); err != nil {
klog.V(1).ErrorS(err, "unable to validate plugin arguments", "pluginName", removepodsviolatingtopologyspreadconstraint.PluginName)
Expand Down

0 comments on commit 7349b39

Please sign in to comment.