Fixup e2e test func errorExistsInLog

Author: vishesh92

pkg/cloud/zone.go

@@ -30,7 +30,6 @@ type ZoneIFace interface {
 
 func (c *client) ResolveZone(zSpec *infrav1.CloudStackZoneSpec) (retErr error) {
 	if zoneID, count, err := c.cs.Zone.GetZoneID(zSpec.Name); err != nil {
-		retErr = multierror.Append(retErr, errors.Wrapf(err, "could not get Zone ID from %v", zSpec.Name))
 		c.customMetrics.EvaluateErrorAndIncrementAcsReconciliationErrorCounter(err)
 		retErr = multierror.Append(retErr, errors.Wrapf(err, "could not get Zone ID from %v", zSpec.Name))
 	} else if count != 1 {

test/e2e/invalid_resource.go

@@ -22,6 +22,7 @@ import (
 	"fmt"
 	"os"
 	"path/filepath"
+	"regexp"
 	"strings"
 
 	. "github.com/onsi/ginkgo/v2"
@@ -46,6 +47,7 @@ var (
 	cancelWatches         context.CancelFunc
 	clusterResources      *clusterctl.ApplyClusterTemplateAndWaitResult
 	MachineInErrorMessage = "CloudStackMachine VM in error state. Deleting associated Machine"
+	errorLogRe            = regexp.MustCompile(`(?m)^([IWEF]\d{4} \d{2}:\d{2}:\d{2}\.\d+.+?)(?=^[IWEF]\d{4} \d{2}:\d{2}:\d{2}\.\d+|$)`)
 )
 
 // InvalidResourceSpec implements a test that verifies that creating a new cluster fails when the specified resource does not exist
@@ -224,13 +226,21 @@ func errorExistsInLog(logFolder string, expectedError string) (bool, error) {
 		}
 
 		if strings.Contains(path, "manager.log") {
-			log, _ := os.ReadFile(path)
-			logLines := strings.Split(string(log), "\n")
-			for _, line := range logLines {
-				if strings.Contains(line, expectedError) &&
-					strings.Contains(line, clusterResources.Cluster.Namespace) {
-					Byf("Found %q error", expectedError)
-					return expectedErrorFound
+			logContent, err := os.ReadFile(path)
+			if err != nil {
+				return err
+			}
+
+			matches := errorLogRe.FindAllStringSubmatch(string(logContent), -1)
+
+			for _, match := range matches {
+				if len(match) > 1 {
+					logEntry := match[1]
+					if strings.Contains(logEntry, expectedError) &&
+						strings.Contains(logEntry, clusterResources.Cluster.Namespace) {
+						Byf("Found %q error", expectedError)
+						return expectedErrorFound
+					}
 				}
 			}
 		}