Skip to content

Commit

Permalink
buildscripts: Add missing secondary_kube_context to xds LB tests (grp…
Browse files Browse the repository at this point in the history
…c#9380)

Secondary cluster was not activated for LB tests. This resulted in the failover test failing, as it relies on workloads running in different zones.

ref b/238226704
  • Loading branch information
sergiitk authored and temawi committed Jul 22, 2022
1 parent d9d8a84 commit fecbef0
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions buildscripts/kokoro/xds_k8s_lb.sh
Original file line number Diff line number Diff line change
Expand Up @@ -103,6 +103,7 @@ build_docker_images_if_needed() {
# Globals:
# TEST_DRIVER_FLAGFILE: Relative path to test driver flagfile
# KUBE_CONTEXT: The name of kubectl context with GKE cluster access
# SECONDARY_KUBE_CONTEXT: The name of kubectl context with secondary GKE cluster access, if any
# TEST_XML_OUTPUT_DIR: Output directory for the test xUnit XML report
# SERVER_IMAGE_NAME: Test server Docker image name
# CLIENT_IMAGE_NAME: Test client Docker image name
Expand All @@ -121,6 +122,7 @@ run_test() {
python -m "tests.${test_name}" \
--flagfile="${TEST_DRIVER_FLAGFILE}" \
--kube_context="${KUBE_CONTEXT}" \
--secondary_kube_context="${SECONDARY_KUBE_CONTEXT}" \
--server_image="${SERVER_IMAGE_NAME}:${GIT_COMMIT}" \
--client_image="${CLIENT_IMAGE_NAME}:${GIT_COMMIT}" \
--testing_version="${TESTING_VERSION}" \
Expand Down Expand Up @@ -158,6 +160,7 @@ main() {
source /dev/stdin <<< "$(curl -s "${TEST_DRIVER_INSTALL_SCRIPT_URL}")"

activate_gke_cluster GKE_CLUSTER_PSM_BASIC
activate_secondary_gke_cluster GKE_CLUSTER_PSM_LB

set -x
if [[ -n "${KOKORO_ARTIFACTS_DIR}" ]]; then
Expand Down

0 comments on commit fecbef0

Please sign in to comment.