cronでpythonの重複起動を禁止したい

前提

さくらレンタルサーバースタンダードプラン
cronでスクリプト実行

参考サイト
https://qiita.com/dev_colla/items/6ab765fc6daf2c2a2baa

実現したいこと

cronでスクリプトの重複起動を禁止したいと思っています。
sshからスクリプトを実行すると重複起動が禁止されているのを確認できますがcronから実行すると
スクリプトが実行していない状態でも重複判定されて実行できません。
何が原因なのでしょうか？

該当のソースコード

import discord
from discord.ext import commands
# 2重起動を防ぐ
from os import path
import subprocess

file_name = path.basename(__file__)
p1 = subprocess.Popen(["ps", "-ef"], stdout=subprocess.PIPE)
p2 = subprocess.Popen(["grep", file_name], stdin=p1.stdout, stdout=subprocess.PIPE)
p3 = subprocess.Popen(["grep", "python"], stdin=p2.stdout, stdout=subprocess.PIPE)
p4 = subprocess.Popen(["wc", "-l"], stdin=p3.stdout, stdout=subprocess.PIPE)
p1.stdout.close()
p2.stdout.close()
p3.stdout.close()
output = p4.communicate()[0].decode("utf8").replace('\n','')

if int(output) != 1:
    print("重複起動")
    exit()

# 実行スクリプト
bot = commands.Bot(command_prefix="/",intents=discord.Intents.all())

@bot.event
async def on_ready():
    channel_sent = bot.get_channel(1012237139729199136)
    await channel_sent.send("完了")
    

token = 'xxxxxxxxxxxxxxxxxxxxxxxxxxxxx'
bot.run(token)

CHERRY

2022/10/03 09:44

各変数の値を確認する必要があると思います。まずは、 output には何が入っていますか？

行動規範の内容に同意します

回答1件

ベストアンサー

二重起動を防止したい（すでに起動中なら実行しない）ということなら、
まず、プログラムのpsやらgrepやらのあたりを全部削除して、処理本体だけにします。

cronへの登録では、
flock -n /tmp/fooo python /path/to/script/foo.py
のようにflock経由で起動します。/tmp/foooの部分は何でも良いので、
flock -n /path/to/script/foo.py python /path/to/script/foo.py
のようにスクリプトファイル自体を指定すると分かり易いかもしれません。

-nを付けないと「起動中ならそのプロセスが終わるまで待つ」になり、
-nを付けると「起動中ならコマンドを実行しない」になります。

手動での実行時も常にflock経由で実行します。

追記

Pythonのみでやる方法。

Python
1import fcntl
2
3f = open(__file__,"r")
4try:
5    fcntl.flock(f.fileno(), fcntl.LOCK_EX|fcntl.LOCK_NB)
6except BlockingIOError:
7    print("重複起動")
8    exit()