Fix sidecarset inconsistentStatus always True bug when restart, close #…

…1636 (#1641) Signed-off-by: Spground <pingshan.wj@alibaba-inc.com> Co-authored-by: Spground <pingshan.wj@alibaba-inc.com>
openkruise · Jun 12, 2024 · 8e2f8f5 · 8e2f8f5
1 parent 5ea03f1
commit 8e2f8f5
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/pkg/controller/sidecarset/sidecarset_processor.go b/pkg/controller/sidecarset/sidecarset_processor.go
@@ -626,7 +626,7 @@ func inconsistentStatus(sidecarSet *appsv1alpha1.SidecarSet, status *appsv1alpha
 		status.ReadyPods != sidecarSet.Status.ReadyPods ||
 		status.UpdatedReadyPods != sidecarSet.Status.UpdatedReadyPods ||
 		status.LatestRevision != sidecarSet.Status.LatestRevision ||
-		status.CollisionCount != sidecarSet.Status.CollisionCount
+		!pointer.Int32Equal(sidecarSet.Status.CollisionCount, status.CollisionCount)
 }
 
 func isSidecarSetUpdateFinish(status *appsv1alpha1.SidecarSetStatus) bool {