Как запретить одновременный запуск PHP скрипта по крону?

Question

[Кудрин Сергей]

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

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

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

Вспомнил про flock. Накидал для проверки вот такой вариант:

$lockfile = fopen("my_script.lock", 'w');
if(flock($lockfile, LOCK_EX | LOCK_NB ) !== true)
	die("already running");
echo 'Good';
flush();
sleep(10);

Запускаю через браузер 1-ый раз скрипт - пошел в работу, во второй вкладке запускаю его еще раз и вместо ожидаемого "already running" вижу, что скрипт встал в очередь и ждет пока завершится предыдущий, после завершения 1-го, отрабатывает 2-ой.

Искал другие варианты реализации на flock - результаты аналогичные, второй запуск скрипта приводит к ожиданию завершения первого.

Мне такой вариант не подходит, так как я не понимаю, что произойдет, если скрипт будет выполняться, например за 3 минуты, а вызываться по крону будет каждую минуту - сформируется длинная очередь ожидающих?

Подскажите, пожалуйста, какое-нибудь максимально универсальное решение.

Answer 1

[DevMan]

$lock = fopen('/tmp/lock_file.lock', 'w');
if ( !($lock && flock($lock, LOCK_EX | LOCK_NB)) ) {
    exit( 'already running' );
}

Comments

[Кудрин Сергей]

А чем это отличается от варианта, что я написал в посте?)

[DevMan]

Кудрин Сергей: а вы сами не видите?

[DevMan]

Кудрин Сергей: я просто скопипастил то, чем сам постоянно пользуюсь.
кстати, ваш вариант у меня тоже работает.

[Кудрин Сергей]

Ну у меня ваш вариант работает так же, как и мой ) Они одинаковые по сути. Первый скрипт уходит в работу, а вторая версия висит в ожидании завершения первого.

[DevMan]

Кудрин Сергей: походу у вас не срабатывает блок if.
проверьте как будет работать с указанием полного пути к лок-файлу.

[Кудрин Сергей]

Если я запускаю такую же проверку, только из другого php файла - то всё работает. Я этого понять не могу. А если запускать тот же, то вот такая фигня. (проверил на разных хостингах)

С полными путями так же

[DevMan]

Кудрин Сергей: я одного понять не могу: зачем вы запускаете их в браузере когда этот режим предназначен для консоли?

[Кудрин Сергей]

Ну я не думал, что будет разница между запуском с браузера и wget'ом с консоли. Она есть?

[DevMan]

Кудрин Сергей: зачем вообще пускать кроном через wget? это же жопорукий метод.
что-то вроде /usr/bin/php /path/to/script - и всех делов.

[DevMan]

Кудрин Сергей: вот объяснение вашей ботвы, если с английским дружите - stackoverflow.com/a/21163983
и в подтверждение написаного, если дернуть код одновременно из разных браузеров, то он работает как ожидается.

[Кудрин Сергей]

DevMan: я всё понял. Сглупил, спасибо за помощь!

[DevMan]

Кудрин Сергей: пожалуйста)

Answer 2

[maximw]

Если в скрипте уже используется коннект к Mysql, можно еще использовать его реализацию семафора, не "загрязняя" файловую систему.
Посмотрите на функцию: GET_LOCK()
Этот метод лишен недостатка, описанного вами. При "умирании" процесса который сделал лок, лок освобождается автоматически.

P,S, Если уж используете файлы, посмотрите register_shutdown_function(), где можно аварийно удалять лок-файл.

Comments

[DevMan]

в способе автора лок также освобождается автоматически.
там проблема в другом.

Answer 3

[entermix]

$fh = fopen(__FILE__, 'r');
        if (!flock($fh, LOCK_EX | LOCK_NB)) {
            die('Script is already running!');
        }

Answer 4

[ShamblerR]

лок есть в самом кроне