fix check for systemd-timesyncd.service

kola/tests/bpf/local-gadget.go

@@ -123,7 +123,7 @@ func localGadgetTest(c cluster.TestCluster) {
 
 	tmpl, err := template.New("user-data").Parse(config)
 	if err != nil {
-		c.Fatalf("parsing user-data: %w", err)
+		c.Fatalf("parsing user-data: %v", err)
 	}
 
 	c.Run("dns gadget", func(c cluster.TestCluster) {
@@ -133,7 +133,7 @@ func localGadgetTest(c cluster.TestCluster) {
 
 		var buf bytes.Buffer
 		if err := tmpl.Execute(&buf, gadget); err != nil {
-			c.Fatalf("rendering user-data: %w", err)
+			c.Fatalf("rendering user-data: %v", err)
 		}
 
 		node, err := c.NewMachine(conf.ContainerLinuxConfig(buf.String()))

kola/tests/coretest/core.go

@@ -322,27 +322,38 @@ func TestServicesActive() error {
 	return servicesActive([]string{
 		"multi-user.target",
 		"docker.socket",
-		"systemd-timesyncd.service",
 		"update-engine.service",
+	}, []string{
+		"systemd-timesyncd.service",
+		"chronyd.service",
+		"ntpd.service",
 	})
 }
 
 func TestServicesActiveCoreOS() error {
 	return servicesActive([]string{
 		"multi-user.target",
-	})
+	}, []string{})
 }
 
-func servicesActive(units []string) error {
+func servicesActive(allOf []string, anyOf []string) error {
 	//t.Parallel()
-	for _, unit := range units {
+	for _, unit := range allOf {
 		c := exec.Command("systemctl", "is-active", unit)
 		err := c.Run()
 		if err != nil {
-			return fmt.Errorf("Services Active: %v", err)
+			return fmt.Errorf("services Active: %s: %v", unit, err)
 		}
 	}
-	return nil
+	var err error
+	for _, unit := range anyOf {
+		c := exec.Command("systemctl", "is-active", unit)
+		err = c.Run()
+		if err == nil {
+			return nil
+		}
+	}
+	return fmt.Errorf("no NTP service active: %v", err)
 }
 
 func TestServicesDisabledRHCOS() error {

kola/tests/ostree/basic.go

@@ -212,7 +212,7 @@ func ostreeRemoteTest(c cluster.TestCluster) {
 		osRemoteListSplit := strings.Split(string(osRemoteListOut), "\n")
 		// should have original remote + newly added remote
 		if len(osRemoteListSplit) != initialRemotesNum+1 {
-			c.Fatalf(`Did not find expected amount of ostree remotes: %q. Expected %d`, string(osRemoteListOut), osRemoteListSplit)
+			c.Fatalf(`Did not find expected amount of ostree remotes: %q. Expected %d`, string(osRemoteListOut), len(osRemoteListSplit))
 		}
 
 		var remoteFound bool = false

kola/tests/ostree/unlock.go

@@ -252,7 +252,7 @@ func ostreeHotfixTest(c cluster.TestCluster) {
 		}
 
 		if rollbackStatus.Deployments[0].Unlocked != "none" {
-			c.Fatalf(`Rollback did not remove hotfix mode; got: $q`, rollbackStatus.Deployments[0].Unlocked)
+			c.Fatalf(`Rollback did not remove hotfix mode; got: %q`, rollbackStatus.Deployments[0].Unlocked)
 		}
 
 		_, secCmdErr := c.SSH(m, ("command -v " + rpmName))

kola/tests/rpmostree/deployments.go

@@ -330,7 +330,7 @@ func rpmOstreeInstallUninstall(c cluster.TestCluster) {
 
 		// check the metadata to make sure everything went well
 		if len(postUninstallStatus.Deployments) != 2 {
-			c.Fatal("Expected %d deployments, got %d", 2, len(postUninstallStatus.Deployments))
+			c.Fatalf("Expected %d deployments, got %d", 2, len(postUninstallStatus.Deployments))
 		}
 
 		if postUninstallStatus.Deployments[0].Checksum != originalCsum {