基于server.log日志的WildFly智能启停脚本问题及优化方案咨询

我来帮你分析下原脚本的问题，然后给出更可靠的实现方案——你遇到的grep+tail在函数里失效的情况，主要是因为进程环境差异和缺少超时控制，同时原脚本还有一些语法和逻辑上的小问题。

问题分析 #

1. 日志监听失效：原脚本里的grep -m 1 "stopped in" <(tail -n 0 -f ...)在函数中不生效，大概率是因为systemctl stop wildfly执行后，WildFly的日志写入进程很快终止，函数环境下的tail -f子进程还没来得及捕获到最后一行日志就被回收了；而单独运行脚本时，进程环境更独立，tail -f有足够时间抓到日志。
2. 语法错误：原脚本里的rcr = $(expr $rcr + 1)有语法问题——bash变量赋值不能有空格，应该写成rcr=$(expr $rcr + 1)。
3. 逻辑漏洞：原循环while [ "$rcr" -eq 0 ]会在启动成功后一直执行停止操作，这显然不符合预期，而且没有重试次数限制，可能导致死循环。

优化后的脚本实现 #

下面是重构后的脚本，解决了上述问题，同时增加了超时控制、重试机制和更清晰的日志输出：

# 通用函数：等待日志中出现指定关键词，支持超时设置
wait_for_log() {
    local log_file="$1"
    local keyword="$2"
    local timeout_seconds="${3:-30}"  # 默认超时30秒

    # 用timeout限制等待时长，tail -f实时监听日志，grep匹配到关键词立即退出
    timeout "$timeout_seconds" grep -m 1 "$keyword" <(tail -n 0 -f "$log_file")
    return $?
}

# 停止WildFly并等待停止成功日志
stopwf() {
    echo "[INFO] Stopping WildFly service..."
    systemctl stop wildfly
    # 等待"stopped in"日志，超时30秒
    wait_for_log "/opt/wildfly/standalone/log/server.log" "stopped in" 30
    return $?
}

# 启动WildFly并等待启动成功日志
startwf() {
    echo "[INFO] Starting WildFly service..."
    systemctl start wildfly
    # 启动可能较慢，设置60秒超时
    wait_for_log "/opt/wildfly/standalone/log/server.log" "started in" 60
    return $?
}

# 主逻辑
if stopwf; then
    echo "[SUCCESS] WildFly stopped properly. Starting service..."
    if startwf; then
        echo "[SUCCESS] WildFly started successfully!"
        exit 0
    else
        echo "[ERROR] Failed to start WildFly after successful stop!"
        exit 1
    fi
else
    echo "[WARNING] Did not detect 'stopped in' log. Starting WildFly for recovery..."
    # 先尝试启动服务
    if ! startwf; then
        echo "[ERROR] Failed to start WildFly during recovery!"
        exit 1
    fi
    # 尝试停止，最多重试3次
    local retry=0
    local max_retries=3
    while [ $retry -lt $max_retries ]; do
        echo "[INFO] Attempting to stop WildFly (retry $((retry+1))/$max_retries)..."
        if stopwf; then
            echo "[SUCCESS] WildFly stopped successfully after recovery start!"
            exit 0
        fi
        retry=$((retry+1))
        sleep 2  # 重试前短暂等待，给服务稳定时间
    done
    echo "[ERROR] Failed to stop WildFly after $max_retries retries!"
    exit 1
fi

优化点说明 #

• 复用的日志监听函数：把日志检测逻辑封装成wait_for_log，可以在启停操作中复用，减少冗余代码。
• 超时控制：用timeout命令限制日志等待时间，避免脚本无限挂起。
• 清晰的状态反馈：增加了[INFO]/[SUCCESS]/[ERROR]前缀的日志输出，方便排查问题。
• 重试机制：在恢复流程中增加了最多3次停止重试，避免单次偶发失败导致脚本终止。
• 变量作用域优化：使用local关键字声明函数内的局部变量，避免全局变量污染。

为什么原命令单独运行有效？ #

当你把grep -m 1 "stopped in" <(tail -n 0 -f /opt/wildfly/standalone/log/server.log)单独写成脚本运行时，该进程是独立的，不会受到原脚本函数执行结束后的进程回收影响，所以tail -f有足够时间捕获到WildFly停止时的日志输出。而在原脚本的函数中，函数执行完成后，其创建的子进程（包括tail -f）可能会被提前终止，导致没抓到日志。

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