Skip to content

Commit

Permalink
Merge pull request #34647 from zmerlynn/fix-kubemark-log-copy
Browse files Browse the repository at this point in the history
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
  • Loading branch information
Kubernetes Submit Queue committed Oct 13, 2016
2 parents eed41cd + 98bcb69 commit a3603c9
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions cluster/log-dump.sh
Original file line number Diff line number Diff line change
Expand Up @@ -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)
Expand All @@ -120,7 +120,7 @@ function save-logs() {
fi
else
case "${KUBERNETES_PROVIDER}" in
gce|gke)
gce|gke|kubemark)
files="${files} ${gce_logfiles}"
;;
aws)
Expand Down

0 comments on commit a3603c9

Please sign in to comment.