From 4ab3103293f5032672c2dbdd00666421a0882c51 Mon Sep 17 00:00:00 2001 From: Jan Tattermusch Date: Wed, 11 May 2022 17:12:45 +0200 Subject: [PATCH] Upload results from bazel C/C++ ARM64 tests to bigquery (#29619) * upload RBE results for linux arm64 tests * propagate UPLOAD_TEST_RESULTS env variable * improve arm64 c/c++ job config * add comment --- tools/internal_ci/linux/arm64/grpc_bazel.sh | 3 ++- .../linux/arm64/grpc_bazel_test_c_cpp.cfg | 16 ++++++++++++++++ 2 files changed, 18 insertions(+), 1 deletion(-) diff --git a/tools/internal_ci/linux/arm64/grpc_bazel.sh b/tools/internal_ci/linux/arm64/grpc_bazel.sh index 60182ac7711..da3e9cb530b 100755 --- a/tools/internal_ci/linux/arm64/grpc_bazel.sh +++ b/tools/internal_ci/linux/arm64/grpc_bazel.sh @@ -22,6 +22,7 @@ source $(dirname $0)/../../../../tools/internal_ci/helper_scripts/move_src_tree_ cd $(dirname $0)/../../../.. source tools/internal_ci/helper_scripts/prepare_build_linux_rc - export DOCKERFILE_DIR=tools/dockerfile/test/bazel_arm64 +# propagate the UPLOAD_TEST_RESULTS env variable to the docker container +export EXTRA_DOCKER_ARGS="-e=UPLOAD_TEST_RESULTS" exec tools/run_tests/dockerize/build_and_run_docker.sh "${BAZEL_SCRIPT}" diff --git a/tools/internal_ci/linux/arm64/grpc_bazel_test_c_cpp.cfg b/tools/internal_ci/linux/arm64/grpc_bazel_test_c_cpp.cfg index 2511409ceba..9a4ea40a07a 100644 --- a/tools/internal_ci/linux/arm64/grpc_bazel_test_c_cpp.cfg +++ b/tools/internal_ci/linux/arm64/grpc_bazel_test_c_cpp.cfg @@ -24,7 +24,23 @@ action { } } +# necessary for the upload_rbe_results.py to be able to fetch the test results that will be upload to bigquery. +gfile_resources: "/bigstore/grpc-testing-secrets/gcp_credentials/resultstore_api_key" + +bazel_setting { + # In order for Kokoro to recognize this as a bazel build and publish the bazel resultstore link, + # the bazel_setting section needs to be present and "upsalite_frontend_address" needs to be + # set. The rest of configuration from bazel_setting is unused (we configure everything when bazel + # command is invoked). + upsalite_frontend_address: "https://source.cloud.google.com" +} + env_vars { key: "BAZEL_SCRIPT" value: "tools/internal_ci/linux/arm64/grpc_bazel_test_c_cpp_in_docker.sh" } + +env_vars { + key: "UPLOAD_TEST_RESULTS" + value: "true" +}