review comments

tools/bazelify_tests/grpc_run_bazel_distribtest_test.sh

@@ -28,8 +28,8 @@ REPORT_SUITE_NAME="$(echo ${TEST_TARGET} | sed 's|^.*[:/]||')"
 tar -xopf ${ARCHIVE_WITH_SUBMODULES}
 cd grpc
 
-# Remove the override the "do not detect toolchain" setting that was set
-# by the from .bazelrc configuration for the remote build.
+# Override the "do not detect toolchain" setting that was set
+# by the .bazelrc configuration for the remote build.
 # TODO(jtattermusch): find a better solution to avoid breaking toolchain detection.
 export BAZEL_DO_NOT_DETECT_CPP_TOOLCHAIN=0
 

tools/bazelify_tests/grpc_run_simple_command_test.sh

@@ -22,8 +22,8 @@ shift
 tar -xopf ${ARCHIVE_WITH_SUBMODULES}
 cd grpc
 
-# Remove the override the "do not detect toolchain" setting that was set
-# by the from .bazelrc configuration for the remote build.
+# Override the "do not detect toolchain" setting that was set
+# by the .bazelrc configuration for the remote build.
 # TODO(jtattermusch): find a better solution to avoid breaking toolchain detection.
 export BAZEL_DO_NOT_DETECT_CPP_TOOLCHAIN=0
 

tools/bazelify_tests/test/bazel_build_with_grpc_no_xds_negative_test_linux.sh

@@ -15,10 +15,10 @@
 
 set -ex
 
-# Test that builds that need xDS do not build with --define=grpc_no_xds=true
+# Ensure that tests that require XDS do not build with --define=grpc_no_xds=true
 EXIT_CODE=0
 bazel build //test/cpp/end2end/xds:xds_end2end_test --define=grpc_no_xds=true || EXIT_CODE=$?
 if [ $EXIT_CODE -eq 0 ]; then
-  echo "Building xds_end2end_test succeeded even with --define=grpc_no_xds=true"
+  echo "FAILED: Building xds_end2end_test succeeded even with --define=grpc_no_xds=true"
   exit 1
 fi