connectivity/tests: omit vendor name from test name

Construct the pod-to-cidr subtest name from the generic stringh "world" and the IP that is actually used, e.g. using the external IP 1.2.3.4 would lead to the test being named "world-1234". Fixes #1542 Signed-off-by: Tobias Klauser <[email protected]>
cilium · Apr 27, 2023 · 742f724 · 742f724
1 parent 936f61e
commit 742f724
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/connectivity/tests/to-cidr.go b/connectivity/tests/to-cidr.go
@@ -6,6 +6,7 @@ package tests
 import (
 	"context"
 	"fmt"
+	"strings"
 
 	"github.com/cilium/cilium-cli/connectivity/check"
 )
@@ -24,13 +25,11 @@ func (s *podToCIDR) Name() string {
 }
 
 func (s *podToCIDR) Run(ctx context.Context, t *check.Test) {
-	eps := []check.TestPeer{
-		check.HTTPEndpoint("cloudflare-1001", "https://"+t.Context().Params().ExternalOtherIP),
-		check.HTTPEndpoint("cloudflare-1111", "https://"+t.Context().Params().ExternalIP),
-	}
 	ct := t.Context()
 
-	for _, ep := range eps {
+	for _, ip := range []string{ct.Params().ExternalIP, ct.Params().ExternalOtherIP} {
+		ep := check.HTTPEndpoint(fmt.Sprintf("world-%s", strings.ReplaceAll(ip, ".", "")), "https://"+ip)
+
 		var i int
 		for _, src := range ct.ClientPods() {
 			src := src // copy to avoid memory aliasing when using reference