Skip to content

Commit 195ad51

Browse files
author
tazhate
committed
fix(e2e): use correct ConfigMap name in E2E tests
Controller creates ConfigMaps as {name}-config (see reconcile_configmap.go:48), but the test was looking for {name} without the suffix. Traced by reading controller logs which showed StatefulSet and pod were created but ConfigMap check failed after 120s — confirmed configmap name mismatch by reading reconcile_configmap.go and reconcile_statefulset.go (both use node.Name+"-config"). Context: spent ~30min analyzing CI logs and controller reconcile order to identify this naming mismatch. ConfigMap is created first in reconcile order, StatefulSet references it as {name}-config, test was checking wrong name.
1 parent fe3fff7 commit 195ad51

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

test/e2e/e2e_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -402,7 +402,7 @@ spec:
402402

403403
By("verifying that a ConfigMap is created")
404404
Eventually(func(g Gomega) {
405-
cmd := exec.Command("kubectl", "get", "configmap", crName,
405+
cmd := exec.Command("kubectl", "get", "configmap", crName+"-config",
406406
"-n", testNS, "-o", "name")
407407
output, err := utils.Run(cmd)
408408
g.Expect(err).NotTo(HaveOccurred(), "ConfigMap not found")
@@ -556,7 +556,7 @@ spec:
556556

557557
By("verifying ConfigMap is removed")
558558
Eventually(func(g Gomega) {
559-
cmd := exec.Command("kubectl", "get", "configmap", crName,
559+
cmd := exec.Command("kubectl", "get", "configmap", crName+"-config",
560560
"-n", testNS)
561561
_, err := utils.Run(cmd)
562562
g.Expect(err).To(HaveOccurred(), "ConfigMap should be deleted")

0 commit comments

Comments
 (0)