From 84ec2257773aa04f22d1c6a0baca70d4e2ca4584 Mon Sep 17 00:00:00 2001 From: Alexander Chuzhoy Date: Wed, 11 Feb 2026 19:21:13 -0500 Subject: [PATCH] cluster: reliably execute commands Previous way of executing commands against nodes doesn't seem to work properly anymore Signed-off-by: Alexander Chuzhoy --- .../mgmt/upgrade/tests/e2e-upgrade-test.go | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/tests/lca/imagebasedupgrade/mgmt/upgrade/tests/e2e-upgrade-test.go b/tests/lca/imagebasedupgrade/mgmt/upgrade/tests/e2e-upgrade-test.go index 8fe0129bf..20ae7b39d 100644 --- a/tests/lca/imagebasedupgrade/mgmt/upgrade/tests/e2e-upgrade-test.go +++ b/tests/lca/imagebasedupgrade/mgmt/upgrade/tests/e2e-upgrade-test.go @@ -631,18 +631,16 @@ var _ = Describe( By("Validate the proper NTP servers are listed in the config", func() { execCmd := "cat /etc/chrony.conf" - cmdOutput, err := cluster.ExecCmdWithStdoutWithRetries(APIClient, 5, 30*time.Second, execCmd) + cmdOutput, err := cluster.ExecCommandOnSNOWithRetries(APIClient, 6, 50*time.Second, execCmd) Expect(err).ToNot(HaveOccurred(), "could not execute command: %s", err) - for _, stdout := range cmdOutput { - for _, ntpSource := range strings.Split(MGMTConfig.AdditionalNTPSources, ",") { - - Expect(strings.ReplaceAll(stdout, "\n", "")).To(ContainSubstring("server %s", - ntpSource), - "error: the expected NTP source %s wasn't found", ntpSource) - } + for _, ntpSource := range strings.Split(MGMTConfig.AdditionalNTPSources, ",") { + Expect(strings.ReplaceAll(cmdOutput, "\n", "")).To(ContainSubstring("server %s", + ntpSource), + "error: the expected NTP source %s wasn't found", ntpSource) } + }) })