cronに登録するシェルスクリプトについて(スクリプト内でパイプを使いたい)

Question

前提・実現したいこと

CentOS Stram8にて200人規模のファイルサーバを構築しました。
しかしながら、原因不明のファイルロック（DENY_ALL）が発生しており、暫定対策として、smbstatus+grep+killコマンドでシェルスクリプト作成し、cronで定期確認させる事で、
DENY_ALLが発生したPIDを自動的にkillさせたいと考え、qiitaに掲載のあった情報を参考に簡易的なシェルスクリプトを作成しましたが、cronでパイプコマンドが使用できず
smbstatusのみ実行する状況です。
cronの登録方法の修正や、シェルスクリプトの修正などお力沿いお願いいたします。

発生している問題・エラーメッセージ

cron動作の結果を確認すると、シェルスクリプトに記載のsmbstatusのみ実行されパイプ以降のコマンドが実行されない。

シェルスクリプト

 #!/bin/bash
  smbstatus | grep DENY_NONE | awk '{ print "kill -9", $1 }' | sh

cron登録方法

*/1 * * * * /usr/local/samba/bin/smbstatus /home/fs-dentsu03/maintenance/smbstatus.sh

Answer 1

"kill -9"

と書いてますが、これは危険です。kill -9というのはプロセスに対して、PCで言うといきなり電源プラグを抜くような物で、必要な終了時処理が行われませんので、samba全体がおかしくなる可能性があります。
デフォルトのkillで終了させると、必要な終了時処理がちゃんとなされます。

killでプロセスが終了しないようなケースだと、samba全体を再起動した方が良いでしょうね。

質問への回答は、itagakiさんの通りです。

Comments

[kamekame0505]

回答ありがとうございます。
kill -9にしていた理由は、運用支障出ており他ユーザーへの影響が出ないよう強制して削除しておりました。
killでも問題ないことが確認できたため、プログラム側は修正済みです。

Answer 2

*/1 * * * * /usr/local/samba/bin/smbstatus /home/fs-dentsu03/maintenance/smbstatus.sh

これだとコマンドとして /usr/local/samba/bin/smbstatus を実行しますよね。
その引数として/home/fs-dentsu03/maintenance/smbstatus.shを与えてることになりますけど…間違いですよね？

*/1 * * * * /home/fs-dentsu03/maintenance/smbstatus.sh

これでいいのでは。

あと、蛇足になるかもですけど、smbstatus.sh内のコマンドのパスはフルパスにしたほうが。
rootの環境変数PATHがどうなっているかによりますけどね。
crontabには環境変数はほとんど設定されていませんので、/etc/crontabにPATHの設定を記述するか、フルパスで指定する必要があります。

Comments

[kamekame0505]

回答ありがとうございます。
sambaはソースコードからコンパイルしているため、/usr/local/samba/bin/smbstatusを記載しないと
smbstatusコマンドが見つかりませんとcron結果に出るため先頭に記載していました。

rootにsmbstatusの環境変数PATHを指定出来ていないということでしょうか？

[kamekame0505]

smbstatusのPATHですが「/usr/local/samba/bin/smbstatus」になります。
cron実行時のPATHと手動(echoコマンド)実行時のPATHで差分がありましたがこれが原因でしょうか？
cron実行時：/usr/local/samba/bin:/usr/local/samba/sbin:/usr/local/samba/bin:/usr/local/samba/sbin:/usr/local/sbin:/usr/sbin:/usr/bin:/bin:/root/bin
echo実行時：/usr/local/samba/bin:/usr/local/samba/sbin:/home/fs-dentsu03/.local/bin:/home/fs-dentsu03/bin:/usr/local/bin:/usr/bin:/usr/local/sbin:/usr/sbin

[itagagaki]

> smbstatusコマンドが見つかりませんとcron結果に出る
> rootにsmbstatusの環境変数PATHを指定出来ていないということでしょうか？

そうですね。ちょっと私も勘違いがありましたが、cron実行時にはデフォルトでは環境変数PATHはそのようには設定されていません。設定することもできるのですが、とにかくcrontabにはフルパスで書きましょう。

で、私の回答の意味はわかりましたか？
実行したいのはsmbstatus単体ではなくてsmbstatus.shですよね？
あなたのcrontabではsmbstatus単体が実行されているだけです。
smbstatus.shは実行されていません。

[kamekame0505]

コメントが遅くなり申し訳ありません。
回答の意図理解できました。ご指摘頂いた通り、フルパスで記載しましたら問題なく実行できました。
ありがとうございます。