Bash脚本和cron异常

Question

我有一个bash脚本，我运行以检查我的程序是否已挂起，以及是否已将其删除。 如果从命令行运行该脚本工作正常，但如果我使用cron安排它，它会做一些非常奇怪的事情。

基本上，脚本（下面）获取我的程序的PID，并从/ proc /目录中的条目获取其创建的日期/时间。 然后它从系统获取当前日期/时间，并在自1970年以来使用“date”命令将这两个值转换为秒，然后再减去两者。 这通常最终总计2100秒或类似的东西，相当于35分钟。

#!/bin/bash
THEDATE=$(date +%s)
MYPID=$(ps aux|grep -v grep|egrep "MyProgram.exe"|awk '{print $2}')
if (( ${#MYPID} > 0 )); then
    STARTTIME=$(ls -ld /proc/$MYPID|date +%s -d"$(awk '{print $6, $7}')")
    TOTALMINS=$(( ($THEDATE - $STARTTIME) / 60  ))
    if (( $TOTALMINS >= 30 )); then
        kill -9 $MYPID
        logger -t "[KillLongRunningProcesses] Killed my program which had been running for $TOTALMINS minutes"
    fi
fi

从命令行运行时，两个日期变量（THEDATE和STARTTIME）都获得正确的值。 但是当由cron运行时，STARTTIME是错误的。 它有正确的日期，但似乎忽略时间部分并将其设置为午夜，即获得“2009-12-14 00:00:00”而不是“2009-12-14 13:23:00”，其中扔掉所有的计算。

有任何想法吗？ 谢谢。

Answer 1

这就是为什么你不能依赖解析ls 。 如果你的系统有它，你应该使用stat 。

stat --printf=%Y /proc/$MYPID

如果没有，也许您的find可以为您做到：

find /proc -maxdepth 1 -name $MYPID -printf "%T@"

Answer 2

首先关闭永远不会解析ls的输出，阅读这个以了解原因。 接下来，使用pgrep而不是使用awk从“ps aux”上的grep解析PID，可以大大提高脚本。 此外，在您返回多个PID的情况下，您的脚本会崩溃。 最后，在编写shell脚本时尽量不要使用CAPITALS作为变量名; 该约定保留用于export到环境中的变量。

以下脚本试图解决上述问题。 它尽可能高效，它可以处理多个PID的情况。 它还检查以确保在我们杀死之前PID仍然存在，因为当我们杀死父级时它可能会取出子进程。

#!/bin/bash

prog_name="MyProgram.exe"
the_date=$(date +%s)
my_pids=( $(pgrep "$prog_name") )

for ((i=0; i < ${#my_pids[@]}; i++)); do
    if [[ -d /proc/${my_pids[i]} ]]; then
        start_time=$(stat --printf=%Y /proc/${my_pids[i]})
        total_mins=$(( (the_date - start_time) / 60 ))
        if (( $total_mins >= 30 )); then
            kill -9 ${my_pids[i]}
            logger -t "Your custom message here"
        fi
    fi
done

Answer 3

对于它的价值，我常常被PATH和其他未自动设置的环境变量所困扰。

值得一试。 如果不是那么评论，我将删除我的答案。

Answer 4

我已经想出怎么做了。 如果我使用ls的“--time-style = long-iso”参数，则以我需要的格式返回它。 显然，cron对某些命令的偏好与默认用户不同。

我认为Dennis认为“ls”对于解析是不可靠的，但我正在使用的计算机的配置永远不会改变，所以现在它可以工作，我会保持原样。 将来我可能会按照你的方式做事。