Merge pull request #34647 from zmerlynn/fix-kubemark-log-copy

Automatic merge from submit-queue log-dump.sh: Fix kubemark log-dump.sh **What this PR does / why we need it**: Using `log-dump.sh` with the `kubemark` synthetic provider are broken. **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #34446

Merge pull request #34647 from zmerlynn/fix-kubemark-log-copy
a3603c9d · Kubernetes Submit Queue · GitHub · eed41cd6 · 98bcb690 · a3603c9d
Commit a3603c9d authored Oct 12, 2016 by Kubernetes Submit Queue Committed by GitHub Oct 12, 2016
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

log-dump.sh cluster/log-dump.sh +2 -2

No files found.
--- a/cluster/log-dump.sh
+++ b/cluster/log-dump.sh
@@ -96,7 +96,7 @@ function copy-logs-from-node() {
      scp -oLogLevel=quiet -oConnectTimeout=30 -oStrictHostKeyChecking=no -i "${LOG_DUMP_SSH_KEY}" "${LOG_DUMP_SSH_USER}@${node}:${scp_files}" "${dir}" > /dev/null || true
    else
      case "${KUBERNETES_PROVIDER}" in
-        gce|gke)
+        gce|gke|kubemark)
          gcloud compute copy-files --project "${PROJECT}" --zone "${ZONE}" "${node}:${scp_files}" "${dir}" > /dev/null || true
          ;;
        aws)
@@ -120,7 +120,7 @@ function save-logs() {
      fi
    else
      case "${KUBERNETES_PROVIDER}" in
-        gce|gke)
+        gce|gke|kubemark)
          files="${files} ${gce_logfiles}"
          ;;
        aws)