Merge pull request #59486 from soltysh/integration_fixes

Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. Return information about which int tests failed in the summary **What this PR does / why we need it**: Currently if integration tests fail they will print the name of the failing tests as it goes, but the summary only tells that it failed. You need to know what to look for to be able to find all these occurrences. This PR provides a summary next to the failure information, eg: ``` FAILED TESTS: run_cluster_management_tests, run_plugins_tests, ``` Which should greatly simplify developers integration debugging. **Release note**: ```release-note NONE ```

Merge pull request #59486 from soltysh/integration_fixes
c7414323 · Kubernetes Submit Queue · GitHub · aa02c0f5 · 5fec1494 · c7414323
Unverified Commit c7414323 authored Feb 20, 2018 by Kubernetes Submit Queue Committed by GitHub Feb 20, 2018
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 4 deletions

test-cmd-util.sh hack/make-rules/test-cmd-util.sh +4 -4

No files found.
--- a/hack/make-rules/test-cmd-util.sh
+++ b/hack/make-rules/test-cmd-util.sh
@@ -106,7 +106,7 @@ function record_command() {
    juLog -output="${output}" -class="test-cmd" -name="${name}" "$@"
    if [[ $? -ne 0 ]]; then
      echo "Error when running ${name}"
-      foundError="True"
+      foundError="${foundError}""${name}"", "
    fi
    set -o nounset
@@ -4609,7 +4609,7 @@ run_impersonation_tests() {
 # Requires an env var SUPPORTED_RESOURCES which is a comma separated list of
 # resources for which tests should be run.
 runTests() {
-  foundError="False"
+  foundError=""
  if [ -z "${SUPPORTED_RESOURCES:-}" ]; then
    echo "Need to set SUPPORTED_RESOURCES env var. It is a list of resources that are supported and hence should be tested. Set it to (*) to test all resources"
@@ -5108,8 +5108,8 @@ runTests() {
  kube::test::clear_all
-  if [ "$foundError" == "True" ]; then
+  if [ ! -z "${foundError}" ]; then
-    echo "TEST FAILED"
+    echo "FAILED TESTS: ""${foundError}"
    exit 1
  fi
 }