Множественный запуск скрипта без причины, где ошибся?

Question

[Виктор]

Доброго дня, знатоки!

У меня есть телеграмм бот написаный на питоне. В папке расположения бота создал файл crontab.sh со следующим содержимым:

#!/bin/bash
if [ "pgrep -f bot.py" != 0 ]
	then {
		sleep 1  #delay
		/usr/bin/python3.6 /home/usertest/bot.py
		exit 1
	}
else
	{
		echo "Exit! Python bot is already running!"
		exit 1
	}
fi;

файл crontab.sh сделал исполняемым:
chmod u+x /home/usertest/crontab.sh

Далее выполнил под текущим пользователем
crontab -e
и добавил следующую строку:
*/5 * * * * /home/usertest/crontab.sh
по какой-то причине бот запускается в 2-3-4-5 и тд экземплярах. Помогите разобраться
Операционная система ubuntu 18.04

Comments

[Lynn «Кофеман»]

Вот это упячка: if [ "pgrep -f bot.py" != 0 ]
Конечно строка "pgrep -f bot.py" никогда не равна 0

[Vadim Priluzkiy]

Алексей Тен, Да вобще не нужно на неравенство проверять. Если пгреп находит процесс, он выдаст нуль. Достаточно просто if.

[Lynn «Кофеман»]

Вообще похоже на какой-то вотчдог для бедных. Может лучше использовать штаные решения? Upstart/systemd/supervisord и т.п.

[Виктор]

Алексей Тен, можно подробнее?

Answer 1

[Saboteur]

#!/bin/bash
PIDFILE=/home/usertest/bot.pid

if pgrep --pidfile $PIDFILE &>/dev/null; then
    echo "Exit! Python bot is already running!"
    exit 1
else
  nohup /usr/bin/python3.6 /home/usertest/bot.py &>/dev/null &
  echo $!>$PIDFILE
fi

Comments

[Виктор]

спасибо огромное, тоже работает!
А не подскажите, как-то лог можно вести?
Смысл такой: иногда к моему боту кто-то пытается подключиться и результаты этих попыток выводятся в консоль на сервере. А чтобы выводились в какой-то лог файл. Или это лучше со стороны питона копать?

[Saboteur]

nohup /usr/bin/python3.6 /home/usertest/bot.py &>/dev/null &

тут весь вывод консоли мы отправляем в /dev/null
отправляйте его в лог файл

nohup /usr/bin/python3.6 /home/usertest/bot.py &>/home/usertest/bot.log &

[Виктор]

Saboteur, спасибо огроменное!

[Виктор]

Saboteur, день добрый!
бот перезапустился, скрипт отработал как надо но в лог файл ничего не записал. Вот реальный код:

#!/bin/bash
PIDFILE=/home/admin/bot-file.pid

if pgrep --pidfile $PIDFILE &>/dev/null; then
    echo "Exit! Python bot is already running!"
    exit 1
else
  nohup /usr/bin/python3.6 /home/admin/bot-file.py &>/home/admin/bot-file.log &
  echo $!>$PIDFILE
fi

вроде не ошибся

[Saboteur]

еще момент, там рядом нет файлика nohup.out?
nohup по умолчанию может и так перенаправить вывод.

[Виктор]

Saboteur, к сожалению нет. Есть только это:

[Saboteur]

Так если кто-то пытается подключиться, оно сейчас вообще никуда не выводит?

Мы же консоль перенаправляем, в которой раньше что-то было, оно не могло просто исчезнуть.
Если убрать перенаправление и попробовать подключиться, в консоли появляется запись?

[Виктор]

Saboteur, я раньше через консоль запускал скрипт (без крона). И в ней выводилось вот такое:

сейчас консоль вообще пустая. Даже если бот перезапустился, я об этом узнаю из-за строки, которую использую в питоне:

def sendall(text):
    if len(config.users) > 0:
        for user in config.users:
            try:
                bot.send_message(user, text)
            except:
                print()

...
тело бота с функциями
...
sendall('Бот запущен и готов к работе!')
bot.polling()

[Saboteur]

так это у вас вообще ошибки в коде слались, а не полезный лог.
Вы ж хоть прочитайте что там пишет - попытка подключиться к Телеграм API не удалась. То юзер заблокировал, то таймаут.

[Виктор]

Saboteur, ну всё верно, "попытка подключиться к Телеграм API не удалась" - это прокcи не отработал, такое бывает, это "То юзер заблокировал" - юзер левый попытался подключиться к боту и отправить /start. Не просто бот перезупускается иногда, я думал где-то смогу это видеть понять причину. В моем коде (он прост) ошибок нет. Ну я так думаю))) Ну да ладно, самое главное оно работает! В любом случае спасибо!

Answer 2

[Vadim Priluzkiy]

#!/bin/bash
if pgrep -f "/usr/bin/python3.6 /home/usertest/bot.py" >&/dev/null
 then {
    echo "Exit! Python bot is already running!"
    exit 1
  }
else
  {
    sleep 1  #delay
    /usr/bin/python3.6 /home/usertest/bot.py
    exit 0
  }
fi;

Comments

[Виктор]

блин, извиняюсь за невнимательность! Спасибо!

[AUser0]

Чего-то как-то я не понял. Двойные кавычки в bash-е делают запуск программы с аргументами, и возвращают получившийся STDOUT?

Павлины, говоришь? Ххххе-ех! (с) тов. Сухов, "Белое сонце пустыни".

[Vadim Priluzkiy]

Виктор, Я сам накосячил. Обновил до правильного варианта.

[Vadim Priluzkiy]

AUser0, Копипаста зло. ;-)

[Виктор]

Vadim Priluzkiy, а уверены что

if pgrep -f "/usr/bin/python3.6 /home/usertest/bot.py" >&/dev/null

а не
if pgrep -f "/home/usertest/bot.py" >&/dev/null

[Vadim Priluzkiy]

Виктор, Ну в конкретно этом кейсе, включение интерпретатора конечно излишне, но вот, например, если запускаются несколько инстансов одного сервиса(писанного не на питоне / любом другом интерпретируемом, разумеется), с разными параметрами и / или конфигами, полная командная строка бывает очень полезна.

[Saboteur]

А вы уверены, что
>&/dev/null
А не
&> /dev/null?

И зачем в баш использовать {} для if/else блоков и ; в конце fi?

[Vadim Priluzkiy]

Saboteur, {} действительно незачем. Это всё последствия смотрения в код си-подобных языков. ;-) fi не является ошибкой, зато улучшает читаемость. А перенаправление правильное. В полном синтаксисе вы-же пишите 2>&1 а не наоборот.

[Saboteur]

дело в том, что в 2>&1 мы говорим перенаправить второй поток (stderr) в первый (в stdout)

а у вас >&/dev/null
то есть первый поток перенаправить в какой-то поток и имя устройства.
То есть если вы хотите перенаправить именно первый поток в /dev/null, то просто
>/dev/null
Эта запись имеет смысл, что все потоки перенаправляем в /dev/null
&>/dev/null
А вот эта запись >&/dev/null, даже если синтаксически и обрабатывается верно, содержит либо ненужный лишний символ, либо делает что-то не то

[Vadim Priluzkiy]

Saboteur, Она перенаправляет второй поток (STDERR) в первый (STDOUT), Амперсанд именно за это и отвечает. То-есть, грубо говоря это сокращённый синтаксис 2>&1, только без цифр. И да, что мой вариант, что ваш, перенаправляют не все потоки(которых, как известно 9) в первый, а только второй. И что-б закончить наш бесполезный спор, оставлю тут картинку с цитатой из man-а.

[Saboteur]

Ну по дефолту открыты только два out потока, и неважно сколько их есть и сколько открыто, &> работает только с первым и вторым.

Ваши картинки не открываются, стоит использовать habrastorage.

Поэтому процитирую текстом:

There are two formats for redirecting standard output and standard error:
&>word
and
>&word
Of the two forms, the first is preferred. This is semantically equivalent to
>word 2>&1

When using the second form, word may not expand to a number or ‘-’. If it does, other redirection operators apply (see Duplicating File Descriptors below) for compatibility reasons.

Но да, выяснилось что правы оба. Сори.