Bash脚本错误检查正确性咨询：文件传输失败后的临时文件处理

原脚本存在的核心问题 #

• 错误的退出状态判断方式：使用(( kubectl ... ))是算术表达式判断，并非用来检查命令退出状态的正确写法。这种方式会把kubectl的输出当作算术值计算，而非识别命令的退出码（0为成功，非0为失败），会导致语法错误或逻辑判断失效。
• Pod名称获取无效：z_pod=$(kubectl -n get pod)未指定命名空间，且kubectl get pod默认返回所有Pod的完整信息（不是单个Pod名称），后续cp/exec命令会因参数错误直接失败。
• 命名空间参数缺失：所有kubectl命令的-n后未填写具体命名空间，无法定位到目标Pod所在的命名空间。
• 重定向逻辑错误：发送文件命令中的< $file位于双引号外，会被本地Shell解析为读取本地文件，而非在Pod内部使用复制过去的文件。
• 文件路径不匹配：本地临时文件路径$file复制到Pod的/tmp/后，Pod内的文件路径应为/tmp/$(basename $file)，直接用本地路径会导致Pod内找不到文件。

修正后的脚本 #

# 配置目标命名空间
NAMESPACE="your-target-namespace"
# 创建本地临时文件
file=$(mktemp /tmp/file.XXX)

# 删除本地临时文件
rm_tmpfile () { 
    rm -f "$file"
}

# 删除Pod内的临时文件
rm_tmpfilepod () { 
    local pod_file="/tmp/$(basename "$file")"
    kubectl -n "$NAMESPACE" exec "$z_pod" -- rm -f "$pod_file"
}

# 通过标签筛选获取目标Pod名称（根据实际场景调整标签）
z_pod=$(kubectl -n "$NAMESPACE" get pod -l app=your-app-tag -o jsonpath='{.items[0].metadata.name}')

# 检查Pod名称是否获取成功
if [[ -z "$z_pod" ]]; then
    echo "Error: Failed to get target pod name"
    rm_tmpfile
    exit 1
fi

# 复制文件到Pod，失败则报错并清理
if ! kubectl -n "$NAMESPACE" cp "$file" "$z_pod:/tmp/"; then
    echo
    echo "Error: Copying the file to pod failed"
    rm_tmpfilepod
    rm_tmpfile
    exit 2
fi

# 发送文件到服务器，失败则报错并清理
local pod_file="/tmp/$(basename "$file")"
if ! kubectl -n "$NAMESPACE" exec "$z_pod" -- bash -c "/usr/bin/console-producer --bootstrap-server .tunnel.cluster.local:xxxx --topic < '$pod_file'"; then
    echo 
    echo "Error: Sending file to a server failed"
    rm_tmpfilepod
    rm_tmpfile
    exit 3
fi

# 执行成功后清理所有临时文件
rm_tmpfilepod
rm_tmpfile

关键修正说明 #

• 使用if ! 命令; then的方式判断命令失败：!会反转命令退出码的判断逻辑，命令失败时进入错误处理块，完全匹配需求。
• 新增命名空间统一配置：避免重复书写，降低遗漏风险。
• 修正Pod名称获取逻辑：通过标签筛选并提取单个Pod的名称，确保后续命令参数有效。
• 统一Pod内文件路径：用basename获取文件名，保证Pod内的文件路径正确。
• 调整重定向位置：将< '$pod_file'放入bash -c的命令字符串中，确保在Pod内部读取文件。
• 变量加双引号包裹：避免文件名含空格等特殊字符导致的命令错误。
• 新增Pod名称获取失败的处理：提前拦截错误并清理本地临时文件。

内容的提问来源于stack exchange，提问作者expert.free