Bash函数if条件异常：Kubernetes Pod状态检查逻辑错误排查

问题描述 #

我编写了一个Bash函数check_pods_status，用于检查Kubernetes中带swarm标识的Pod状态：仅当所有目标Pod处于Running或Completed状态时正常返回，否则触发错误逻辑并退出。但实际运行时，即使所有Pod都处于Running状态，函数中的if条件仍会执行错误逻辑，请求解决建议。

函数代码如下：

check_pods_status() {
  # Save the output of the kubectl command in an array
  pods_status=`kubectl get pods -o wide | grep "swarm" |  awk '{print $3}'`
  echo "============================================================================================================="
  echo "checking all the microservices in the swarm cluster are running normally ..."
  kubectl get pods -o wide | grep "swarm"
  echo "============================================================================================================="
  for status in "${pods_status[@]}"; do
    if [[ "$status" != "Running" ]] || [[ "$status" != "Completed" ]]; then
      # Print an error message and return 1
      echo "Error: One or more pods are not running or completed"
      return 1
    fi
  done
  # all pods are healthy at this point and function will return 0 whihc is true
  return 0
}

问题集中在以下if条件：

if [[ "$status" != "Running" ]] || [[ "$status" != "Completed" ]]; then

即使Pod状态为Running，该条件仍会触发错误逻辑。我的预期是：当Pod状态既非Running也非Completed时才返回错误。

问题根源 #

这是典型的布尔逻辑错误：||表示逻辑或，只要任意一个条件成立，整个表达式就为真。例如当状态是Running时，[[ "$status" != "Completed" ]]这个条件必然成立，导致整个if表达式结果为真，错误逻辑被触发。

正确的逻辑应该是：当状态既不是Running，同时也不是Completed时才触发错误，此时需要使用逻辑与&&而非逻辑或||。

修复后的代码 #

check_pods_status() {
  # 用mapfile将命令输出逐行存入数组（比反引号更可靠）
  mapfile -t pods_status < <(kubectl get pods -o wide | grep "swarm" | awk '{print $3}')
  echo "============================================================================================================="
  echo "checking all the microservices in the swarm cluster are running normally ..."
  kubectl get pods -o wide | grep "swarm"
  echo "============================================================================================================="
  for status in "${pods_status[@]}"; do
    # 修正逻辑：状态不是Running 且 不是Completed时触发错误
    if [[ "$status" != "Running" ]] && [[ "$status" != "Completed" ]]; then
      echo "Error: One or more pods are not running or completed"
      return 1
    fi
  done
  return 0
}

额外优化建议 #

• 数组存储方式优化：用mapfile -t替代反引号存数组，反引号会将所有输出合并为单个字符串，遍历会按空格分割（若状态含空格会出错），而mapfile会逐行存入数组，更可靠。
• kubectl输出解析优化：可以直接使用kubectl的jsonpath输出，避免依赖grep和awk，防止kubectl输出格式变化导致解析错误：

  mapfile -t pods_status < <(kubectl get pods -o jsonpath='{.items[?(@.metadata.name=~"swarm")].status.phase}')
  

内容的提问来源于stack exchange，提问作者Arjun Singh