Skip to content

Commit

Permalink
ci: use cloud provider specific external target for pod-to-world tests
Browse files Browse the repository at this point in the history
Follows cilium/cilium#23103, quoting its commit
message:

With the configurable --external-target flag we are configuring different targets
per cloud provider. Targets were chosen to be the least hops from the clusters.
They were determined by experimenting with connection times from cloud providers.
Splitting targets also has the advantage of not overloading a single target.

Co-authored-by: Birol Bilgin <[email protected]>
Signed-off-by: Tobias Klauser <[email protected]>
  • Loading branch information
tklauser and brlbil committed Feb 3, 2023
1 parent eeddd57 commit 130289d
Show file tree
Hide file tree
Showing 6 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion .github/in-cluster-test-scripts/aks.sh
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ sleep 10s
[[ $(pgrep -f "cilium.*hubble.*port-forward|kubectl.*port-forward.*hubble-relay" | wc -l) == 2 ]]

# Run connectivity test
cilium connectivity test --debug --all-flows --collect-sysdump-on-failure
cilium connectivity test --debug --all-flows --collect-sysdump-on-failure --external-target bing.com

# Run performance test
cilium connectivity test --perf --perf-duration 1s
Expand Down
2 changes: 1 addition & 1 deletion .github/in-cluster-test-scripts/eks-tunnel.sh
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ sleep 10s
[[ $(pgrep -f "cilium.*hubble.*port-forward|kubectl.*port-forward.*hubble-relay" | wc -l) == 2 ]]

# Run connectivity test
cilium connectivity test --debug --all-flows --collect-sysdump-on-failure \
cilium connectivity test --debug --all-flows --collect-sysdump-on-failure --external-target amazon.com \
--test '!dns-only,!to-fqdns,!client-egress-l7,!health'
# workaround for nslookup issues in tunnel mode causing tests to fail reliably
# TODO: remove once:
Expand Down
2 changes: 1 addition & 1 deletion .github/in-cluster-test-scripts/eks.sh
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ sleep 10s
[[ $(pgrep -f "cilium.*hubble.*port-forward|kubectl.*port-forward.*hubble-relay" | wc -l) == 2 ]]

# Run connectivity test
cilium connectivity test --debug --all-flows --collect-sysdump-on-failure
cilium connectivity test --debug --all-flows --collect-sysdump-on-failure --external-target amazon.com

# Run performance test
cilium connectivity test --perf --perf-duration 1s
2 changes: 1 addition & 1 deletion .github/in-cluster-test-scripts/external-workloads.sh
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ set -x
set -e

# Run connectivity test
cilium connectivity test --debug --all-flows --collect-sysdump-on-failure
cilium connectivity test --debug --all-flows --collect-sysdump-on-failure --external-target google.com

# Run performance test
cilium connectivity test --perf --perf-duration 1s
2 changes: 1 addition & 1 deletion .github/in-cluster-test-scripts/gke.sh
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ sleep 10s
[[ $(pgrep -f "cilium.*hubble.*port-forward|kubectl.*port-forward.*hubble-relay" | wc -l) == 2 ]]

# Run connectivity test
cilium connectivity test --debug --all-flows --collect-sysdump-on-failure
cilium connectivity test --debug --all-flows --collect-sysdump-on-failure --external-target google.com

# Run performance test
cilium connectivity test --perf --perf-duration 1s
2 changes: 1 addition & 1 deletion .github/in-cluster-test-scripts/multicluster.sh
Original file line number Diff line number Diff line change
Expand Up @@ -57,4 +57,4 @@ sleep 10s

# Run connectivity test
cilium --context "${CONTEXT1}" connectivity test --debug --multi-cluster "${CONTEXT2}" --test '!/*-deny,!/pod-to-.*-nodeport' \
--all-flows --collect-sysdump-on-failure
--all-flows --collect-sysdump-on-failure --external-target google.com

0 comments on commit 130289d

Please sign in to comment.