Skip to content

Commit

Permalink
Fix deploy-metallb to pass shellcheck linter
Browse files Browse the repository at this point in the history
  • Loading branch information
leo8a authored and eifrach committed Jun 26, 2022
1 parent 24fbdca commit e78b5c0
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions deploy-metallb/deploy.sh
Original file line number Diff line number Diff line change
Expand Up @@ -192,7 +192,7 @@ function copy_files() {
dst_folder=${3}

if [[ -z ${src_files} ]]; then
echo "Source files variable empty: ${src_files[@]}"
echo "Source files variable empty:" "${src_files[@]}"
exit 1
fi

Expand All @@ -206,8 +206,8 @@ function copy_files() {
exit 1
fi

echo "Copying source files: ${src_files[@]} to Node ${dst_node}"
${SCP_COMMAND} -i ${RSA_KEY_FILE} ${src_files[@]} core@${dst_node}:${dst_folder}
echo "Copying source files:" "${src_files[@]}" "to Node ${dst_node}"
${SCP_COMMAND} -i ${RSA_KEY_FILE} "${src_files[@]}" core@${dst_node}:${dst_folder}
}

function check_external_access() {
Expand Down Expand Up @@ -272,7 +272,7 @@ if ! ./verify.sh; then
# Remote working
echo ">> Copying files to the Edge-cluster ${edgecluster}"
${SSH_COMMAND} -i ${RSA_KEY_FILE} core@${EDGE_NODE_IP} "mkdir -p ~/manifests ~/.kube"
for _file in ${files[@]}; do
for _file in "${files[@]}"; do
copy_files "${_file}" "${EDGE_NODE_IP}" "./manifests/"
done
copy_files "./manifests/*.yaml" "${EDGE_NODE_IP}" "./manifests/"
Expand Down

0 comments on commit e78b5c0

Please sign in to comment.