Skip to content

Commit

Permalink
Bugfix: to-exclude label don't work (#305)
Browse files Browse the repository at this point in the history
  • Loading branch information
ColdsteelRail authored Nov 14, 2024
1 parent 4df7763 commit 1f4d86d
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions pkg/controllers/collaset/synccontrol/scale.go
Original file line number Diff line number Diff line change
Expand Up @@ -117,9 +117,6 @@ func (s ActivePodsForDeletion) Less(i, j int) bool {

// dealIncludeExcludePods returns pods which are allowed to exclude and include
func (r *RealSyncControl) dealIncludeExcludePods(ctx context.Context, cls *appsv1alpha1.CollaSet, pods []*corev1.Pod) (sets.String, sets.String, error) {
if len(cls.Spec.ScaleStrategy.PodToExclude) == 0 && len(cls.Spec.ScaleStrategy.PodToInclude) == 0 {
return nil, nil, nil
}
ownedPods := sets.String{}
excludePodNames := sets.String{}
includePodNames := sets.String{}
Expand Down

0 comments on commit 1f4d86d

Please sign in to comment.