Skip to content

Commit

Permalink
Do not attempt to delete pods multiple times
Browse files Browse the repository at this point in the history
Signed-off-by: Andrea Mazzotti <andrea.mazzotti@suse.com>
  • Loading branch information
anmazzotti committed Sep 26, 2024
1 parent f0c5b11 commit 3e94c75
Showing 1 changed file with 17 additions and 21 deletions.
38 changes: 17 additions & 21 deletions tests/e2e/upgrades_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -129,29 +129,25 @@ var _ = Describe("ManagedOSImage Upgrade e2e tests", Ordered, func() {
k.Delete("managedosversionchannel", "--all", "--wait", "-n", fleetNamespace)

// delete dangling upgrade pods
EventuallyWithOffset(1, func() []string {
pods, err := k.GetPodNames(cattleSystemNamespace, "upgrade.cattle.io/controller=system-upgrade-controller")
if err != nil {
fmt.Println(err)
}
fmt.Println(pods)

applyPods := []string{}
for _, p := range pods {
if !strings.HasPrefix(p, "system-upgrade-controller") {
applyPods = append(applyPods, p)
}

if strings.Contains(p, "apply-os-upgrader") {
By("deleting " + p)
k.Delete("pod", "-n", cattleSystemNamespace, "--wait", "--force", p)
err = k.WaitForPodDelete(cattleSystemNamespace, p)
Expect(err).ToNot(HaveOccurred())
}
pods, err := k.GetPodNames(cattleSystemNamespace, "upgrade.cattle.io/controller=system-upgrade-controller")
if err != nil {
fmt.Println(err)
}
fmt.Println(pods)

applyPods := []string{}
for _, p := range pods {
if !strings.HasPrefix(p, "system-upgrade-controller") {
applyPods = append(applyPods, p)
}

return applyPods
}, 3*time.Minute, 2*time.Second).Should(Equal([]string{}))
if strings.Contains(p, "apply-os-upgrader") {
By("deleting " + p)
k.Delete("pod", "-n", cattleSystemNamespace, "--wait", "--force", p)
err = k.WaitForPodDelete(cattleSystemNamespace, p)
Expect(err).ToNot(HaveOccurred())
}
}
})

createsCorrectPlan := func(meta map[string]runtime.RawExtension, c *upgradev1.ContainerSpec, m types.GomegaMatcher) {
Expand Down

0 comments on commit 3e94c75

Please sign in to comment.