Parcourir la source

e2e: check_adjacent accept node number

This commit modifies the logic of the check_adjacent helper so that
rather than expecting the argument to be n^2+n it expects simply n. This
makes it easier to update the caller when the number of nodes in the
cluster changes.

Signed-off-by: Lucas Servén Marín <lserven@gmail.com>
Lucas Servén Marín il y a 4 ans
Parent
commit
c728870b49
3 fichiers modifiés avec 5 ajouts et 5 suppressions
  1. 2 2
      e2e/full-mesh.sh
  2. 1 1
      e2e/lib.sh
  3. 2 2
      e2e/location-mesh.sh

+ 2 - 2
e2e/full-mesh.sh

@@ -11,10 +11,10 @@ setup_suite() {
 
 
 test_full_mesh_connectivity() {
 test_full_mesh_connectivity() {
 	assert "retry 30 5 '' check_ping" "should be able to ping all Pods"
 	assert "retry 30 5 '' check_ping" "should be able to ping all Pods"
-	assert "retry 10 5 'the adjacency matrix is not complete yet' check_adjacent 12" "adjacency should return the right number of successful pings"
+	assert "retry 10 5 'the adjacency matrix is not complete yet' check_adjacent 3" "adjacency should return the right number of successful pings"
 	echo "sleep for 30s (one reconciliation period) and try again..."
 	echo "sleep for 30s (one reconciliation period) and try again..."
 	sleep 30
 	sleep 30
-	assert "retry 10 5 'the adjacency matrix is not complete yet' check_adjacent 12" "adjacency should return the right number of successful pings after reconciling"
+	assert "retry 10 5 'the adjacency matrix is not complete yet' check_adjacent 3" "adjacency should return the right number of successful pings after reconciling"
 }
 }
 
 
 test_full_mesh_peer() {
 test_full_mesh_peer() {

+ 1 - 1
e2e/lib.sh

@@ -157,7 +157,7 @@ check_ping() {
 
 
 check_adjacent() {
 check_adjacent() {
         _kubectl get pods -l app.kubernetes.io/name=curl -o name | xargs -I{} "$KUBECTL_BINARY" --kubeconfig="$KUBECONFIG" exec {} -- /bin/sh -c 'curl -m 1 -s adjacency:8080/?format=fancy'
         _kubectl get pods -l app.kubernetes.io/name=curl -o name | xargs -I{} "$KUBECTL_BINARY" --kubeconfig="$KUBECONFIG" exec {} -- /bin/sh -c 'curl -m 1 -s adjacency:8080/?format=fancy'
-	[ "$(curl_pod -m 1 -s adjacency:8080/?format=json | jq | grep -c true)" -eq "$1" ]
+	[ "$(curl_pod -m 1 -s adjacency:8080/?format=json | jq '.[].latencies[].ok' | grep -c true)" -eq $(($1*$1)) ]
 }
 }
 
 
 check_peer() {
 check_peer() {

+ 2 - 2
e2e/location-mesh.sh

@@ -11,10 +11,10 @@ setup_suite() {
 
 
 test_location_mesh_connectivity() {
 test_location_mesh_connectivity() {
 	assert "retry 30 5 '' check_ping" "should be able to ping all Pods"
 	assert "retry 30 5 '' check_ping" "should be able to ping all Pods"
-	assert "retry 10 5 'the adjacency matrix is not complete yet' check_adjacent 12" "adjacency should return the right number of successful pings"
+	assert "retry 10 5 'the adjacency matrix is not complete yet' check_adjacent 3" "adjacency should return the right number of successful pings"
 	echo "sleep for 30s (one reconciliation period) and try again..."
 	echo "sleep for 30s (one reconciliation period) and try again..."
 	sleep 30
 	sleep 30
-	assert "retry 10 5 'the adjacency matrix is not complete yet' check_adjacent 12" "adjacency should return the right number of successful pings after reconciling"
+	assert "retry 10 5 'the adjacency matrix is not complete yet' check_adjacent 3" "adjacency should return the right number of successful pings after reconciling"
 }
 }
 
 
 test_location_mesh_peer() {
 test_location_mesh_peer() {