fix sidecarset e2e (#1724)

Signed-off-by: liheng.zms <liheng.zms@alibaba-inc.com>
openkruise · Sep 3, 2024 · 4918768 · 4918768
1 parent 68a3793
commit 4918768
Showing 1 changed file with 15 additions and 6 deletions.
diff --git a/test/e2e/apps/sidecarset.go b/test/e2e/apps/sidecarset.go
@@ -853,10 +853,12 @@ var _ = SIGDescribe("SidecarSet", func() {
 			gomega.Expect(err).NotTo(gomega.HaveOccurred())
 			for _, pod := range pods {
 				origin := sets.String{}
-				for _, sidecar := range sidecarSetIn.Spec.Containers {
-					origin.Insert(sidecar.Name)
-				}
 				for _, sidecar := range sidecarSetIn.Spec.InitContainers {
+					if sidecarcontrol.IsSidecarContainer(sidecar.Container) {
+						origin.Insert(sidecar.Name)
+					}
+				}
+				for _, sidecar := range sidecarSetIn.Spec.Containers {
 					origin.Insert(sidecar.Name)
 				}
 				// SidecarSetHashAnnotation = "kruise.io/sidecarset-hash"
@@ -902,10 +904,12 @@ var _ = SIGDescribe("SidecarSet", func() {
 			gomega.Expect(err).NotTo(gomega.HaveOccurred())
 			for _, pod := range pods {
 				origin := sets.String{}
-				for _, sidecar := range sidecarSetIn.Spec.Containers {
-					origin.Insert(sidecar.Name)
-				}
 				for _, sidecar := range sidecarSetIn.Spec.InitContainers {
+					if sidecarcontrol.IsSidecarContainer(sidecar.Container) {
+						origin.Insert(sidecar.Name)
+					}
+				}
+				for _, sidecar := range sidecarSetIn.Spec.Containers {
 					origin.Insert(sidecar.Name)
 				}
 				// SidecarSetHashAnnotation = "kruise.io/sidecarset-hash"
@@ -950,6 +954,11 @@ var _ = SIGDescribe("SidecarSet", func() {
 			gomega.Expect(err).NotTo(gomega.HaveOccurred())
 			for _, pod := range pods {
 				origin := sets.String{}
+				for _, sidecar := range sidecarSetIn.Spec.InitContainers {
+					if sidecarcontrol.IsSidecarContainer(sidecar.Container) {
+						origin.Insert(sidecar.Name)
+					}
+				}
 				for _, sidecar := range sidecarSetIn.Spec.Containers {
 					origin.Insert(sidecar.Name)
 				}