如何优化Bash脚本以定位含指定Case ID的Docker容器

可以通过循环遍历容器列表并判断日志匹配结果的方式，实现找到匹配日志时输出对应容器名称的需求。修改后的脚本更简洁，也能清晰关联容器与日志：

#!/bin/bash

# 输入Case ID
echo 'Enter Case ID'
read caseID

# 定义需要检查的容器列表
containers=("con1" "con2" "con3" "con4" "con5" "con6")

# 循环检查每个容器
for container in "${containers[@]}"
do
    # 捕获匹配的日志内容
    log_match=$(docker logs "$container" 2>&1 | grep -B 4 "$caseID")
    
    # 如果找到匹配内容，输出容器名和日志
    if [ -n "$log_match" ]; then
        echo "Error in $container"
        echo "$log_match"
        echo "------------------------" # 分隔不同容器的结果
    fi
done

关键修改说明 #

• 容器列表数组：把重复的容器名放到数组里，避免冗余代码，后续新增/删除容器只需修改数组即可。
• 日志捕获与判断：用log_match变量存储每个容器的匹配日志，通过[ -n "$log_match" ]判断是否存在匹配内容（即是否找到目标Case ID）。
• 清晰的结果输出：找到匹配时先打印Error in 容器名，再输出对应日志内容，搭配分隔线区分不同容器的结果，可读性更强。

额外优化建议 #

• 如果不确定容器名称，可以用docker ps --format "{{.Names}}"自动获取运行中的容器名，替换手动定义的数组：

  containers=($(docker ps --format "{{.Names}}"))
  

• 给grep加上-q参数可以只判断是否匹配而不输出内容，再单独输出日志，执行效率更高：

  if docker logs "$container" 2>&1 | grep -q -B 4 "$caseID"; then
      echo "Error in $container"
      docker logs "$container" 2>&1 | grep -B 4 "$caseID"
      echo "------------------------"
  fi
  

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