chore: fixed the conditional statement that determines whether RS is …

…deleted
openkruise · Dec 26, 2023 · f867f18 · f867f18
1 parent df5d596
commit f867f18
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/pkg/controller/deployment/deployment_controller.go b/pkg/controller/deployment/deployment_controller.go
@@ -88,7 +88,7 @@ func (dc *DeploymentController) getReplicaSetsForDeployment(ctx context.Context,
 	// select rs owner by current deployment
 	ownedRSs := make([]*apps.ReplicaSet, 0)
 	for _, rs := range allRSs {
-		if rs.DeletionTimestamp.IsZero() {
+		if !rs.DeletionTimestamp.IsZero() {
 			continue
 		}
 
@@ -116,7 +116,7 @@ func (dc *DeploymentController) syncDeployment(ctx context.Context, deployment *
 	if reflect.DeepEqual(d.Spec.Selector, &everything) {
 		dc.eventRecorder.Eventf(d, v1.EventTypeWarning, "SelectingAll", "This deployment is selecting all pods. A non-empty selector is required.")
 		if d.Status.ObservedGeneration < d.Generation {
-			d.Status.ObservedGene7ration = d.Generation
+			d.Status.ObservedGeneration = d.Generation
 			dc.client.AppsV1().Deployments(d.Namespace).UpdateStatus(ctx, d, metav1.UpdateOptions{})
 		}
 		return nil