fix(backup): 修复本地备份保留修剪静默失效,本地默认保留降到 2 份

bug:prune_local 里 ls 的 cd 只发生在命令替换的子 shell 中,
rm 却拿裸文件名在 PROJECT_ROOT 下执行——目标文件不存在,
而 rm -f 对不存在的文件静默成功。结果是"清理成功"日志照打、
本地备份数量却一直超出 KEEP(生产 configs 目录累积到 43 份,
dump 10 份,均超出 KEEP=7)。

修复:rm 改用 ${dir}/${file} 绝对路径,并补源码级守卫测试锁定。

顺带把 .env.server.example 的默认保留从 7 降到 2:
本地备份只是断网应急,OSS 已有完整备份;而部署脚本的磁盘预检
要求 可用 ≥ 备份目录×2,本地留太多会直接把部署卡死
(这次就是 3.8G 备份触发预检失败 → 被迫清构建缓存 → 全量构建变慢)。
This commit is contained in:
yml2213
2026-10-03 15:51:01 +08:00
parent 69e3d9a396
commit 8ecb8a879a
3 changed files with 27 additions and 3 deletions
+3 -1
View File
@@ -85,7 +85,9 @@ STORAGE_OSS_REGION=oss-cn-hangzhou
# 默认复用上面的 STORAGE_OSS_* 上传到同桶 backups/ 前缀,以下仅需按需调整。 # 默认复用上面的 STORAGE_OSS_* 上传到同桶 backups/ 前缀,以下仅需按需调整。
DB_BACKUP_OSS_PREFIX=backups/postgres DB_BACKUP_OSS_PREFIX=backups/postgres
# 服务器本地保留的备份份数(OSS 端过期请配置桶生命周期规则) # 服务器本地保留的备份份数(OSS 端过期请配置桶生命周期规则)
DB_BACKUP_KEEP=7 # 本地只是断网应急用,OSS 已有完整备份;而 deploy 脚本的磁盘预检要求
# 可用 >= 备份目录x2,本地留太多会把部署卡死(49G 盘建议不超过 2~3 份)。
DB_BACKUP_KEEP=2
# 建议为备份单独创建 RAM 用户、只授予 backups/ 前缀权限(策略见 docs/数据库备份与恢复.md); # 建议为备份单独创建 RAM 用户、只授予 backups/ 前缀权限(策略见 docs/数据库备份与恢复.md);
# 使用独立桶/密钥时取消注释覆盖: # 使用独立桶/密钥时取消注释覆盖:
# DB_BACKUP_OSS_ENDPOINT=https://oss-cn-hangzhou-internal.aliyuncs.com # DB_BACKUP_OSS_ENDPOINT=https://oss-cn-hangzhou-internal.aliyuncs.com
+18
View File
@@ -200,3 +200,21 @@ test('runLatest 跳过没有 manifest 的新对象并选择最近的有效备份
'backups/postgres/valid.dump', 'backups/postgres/valid.dump',
) )
}) })
test('backup-db.sh 的本地保留修剪必须用绝对路径删除(曾静默失效导致备份无限累积)', async () => {
const { readFile } = await import('node:fs/promises')
const script = await readFile(new URL('../../../deploy/backup-db.sh', import.meta.url), 'utf8')
const pruneBody = script.slice(script.indexOf('prune_local() {'), script.indexOf('main() {'))
// 曾经的 bug:cd 只发生在 ls 的命令替换子 shell 里,rm 却拿裸文件名在
// PROJECT_ROOT 下删(目标不存在),rm -f 静默吞错 → 本地备份远超 KEEP 份数。
assert.ok(
pruneBody.includes('rm -f -- "${dir}/${file}"'),
'修剪必须删除 ${dir}/${file};只删文件名会在项目根目录下落空',
)
assert.ok(
!/"\$\{file\}"\s*$/m.test(pruneBody.replace('rm -f -- "${dir}/${file}"', '')),
'不得再出现只按文件名删除的写法',
)
})
+6 -2
View File
@@ -157,8 +157,12 @@ prune_local() {
return return
fi fi
while IFS= read -r file; do while IFS= read -r file; do
rm -f -- "${file}" [[ -n "${file}" ]] || continue
log "清理本地旧备份:${file}(本地保留 ${KEEP} 份,OSS 仍由生命周期规则管理)" # 上面的 cd 只在命令替换的子 shell 里生效,本循环运行时 cwd 是 PROJECT_ROOT。
# 必须拼上目录再删:只删文件名等于在 PROJECT_ROOT 下找同名文件(不存在),
# rm -f 会静默吞掉错误——这正是本地备份数量一直超出 KEEP 的原因。
rm -f -- "${dir}/${file}"
log "清理本地旧备份:${dir}/${file}(本地保留 ${KEEP} 份,OSS 仍由生命周期规则管理)"
done <<<"${removed}" done <<<"${removed}"
} }