古いバックアップファイルを自動削除

Question

以下のスクリプトを毎日実行しているのですが、バックアップはできますが自動削除がされません。
どこがおかしいのでしょうか？

#!/bin/sh

# 他のユーザからバックアップを読み込めないようにする
umask 077

# バックアップファイルを何日分残しておくか（一ヶ月分）
period=5
# バックアップファイルを保存するディレクトリ
dirpath='/root/backup/mysql'

# ファイル名を定義(※ファイル名で日付がわかるようにしておきます)
filename=`date +%Y%m%d`

# mysqldump実行（ファイルサイズ圧縮の為gzで圧縮しておきます。）
mysqldump --opt --all-databases --events --default-character-set=binary -u root --password= | gzip > $dirpath/$filename.sql.gz

# 古いバックアップファイルを削除
oldfile=`date --date "$period days ago" +%Y%m%d`
rm -f $dirpath/$oldfile.sql.gz

Answer 1

削除するコードが入っていないので、削除されません。
rm $oldfile
を最後に追加してください。

あと、コードはコードブロックにしてください。
コード部分を全部選択して、入力欄の上にある「</>」ボタンを押して、「ここに言語を入力」のところを「Bash」に書き直してください。

Comments

[happy_tera]

回答ありがとうございます。
コードブロックにしました。ご指摘ありがとうございます。
最後の一文のrmコマンドに関しては抜けておりまして
rm -f $dirpath/$oldfile.sql.gz
と記述しているにも関わらず削除されるときとされない時があります。

[otn]

削除されないときは、何かエラーメッセージは出ていませんか？
あ、-f が付いていますね。-f は外しておいてください。ファイルが無くてもエラーメッセージが出ないので付けちゃ駄目です。

日をまたがる処理になって、日が時にはずれるという事じゃ無いでしょうね？

[happy_tera]

回答ありがとうございます。
-fを外してエラーメッセージを見てみたいと思います。
毎朝4時にcronで起動しています。

[otn]

ls -l $dirpath
を要所要所に入れておいた方が良いですね。dumpの前後と、rmの前後。