Skip to content

Commit 24d3e18

Browse files
authored
Merge pull request #1645 from chewong/do-not-declare-log-dump-pods
fix: remove log_dump_pods variable declaration
2 parents 31a0eb9 + 7e8c5a3 commit 24d3e18

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

hack/log/log-dump.sh

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -91,8 +91,7 @@ dump_workload_cluster_logs() {
9191
kubectl apply -f "${REPO_ROOT}/hack/log/log-dump-daemonset.yaml"
9292
kubectl wait pod -l app=log-dump-node --for=condition=Ready --timeout=5m
9393

94-
local -r log_dump_pods=()
95-
IFS=" " read -r -a log_dump_pods <<< "$(kubectl get pod -l app=log-dump-node -ojsonpath='{.items[*].metadata.name}')"
94+
IFS=" " read -ra log_dump_pods <<< "$(kubectl get pod -l app=log-dump-node -ojsonpath='{.items[*].metadata.name}')"
9695
local log_dump_commands=(
9796
"journalctl --output=short-precise -u kubelet > kubelet.log"
9897
"journalctl --output=short-precise -u containerd > containerd.log"

0 commit comments

Comments
 (0)