Fix ping checking for memif

Signed-off-by: Artem Glazychev <artem.glazychev@xored.com>
networkservicemesh · Jul 30, 2021 · 70edc83 · 70edc83
1 parent 69a4bb0
commit 70edc83
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/internal/imports/imports_linux.go b/internal/imports/imports_linux.go
diff --git a/internal/tests/suite_memif_test.go b/internal/tests/suite_memif_test.go
@@ -22,7 +22,7 @@ import (
 	"context"
 	"fmt"
 	"net"
-	"regexp"
+	"strings"
 	"time"
 
 	"git.fd.io/govpp.git/api"
@@ -172,7 +172,8 @@ func pingVpp(ctx context.Context, vppConn api.Connection, ipaddress string) erro
 		WithField("Reply", pingRsp.Reply).
 		WithField("duration", time.Since(now)).Debug("completed")
 
-	if regexp.MustCompile(" 0% packet loss").MatchString(pingRsp.Reply) {
+	if strings.Contains(pingRsp.Reply, " 0% packet loss") &&
+		!strings.Contains(pingRsp.Reply, " 0 sent, 0 received") {
 		return nil
 	}
 	return errors.New("Ping failed")