shukivaknin/gniza4linux
1
1
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Treat rsync exit codes 23/24 as warnings, not failures

Browse Source 
Exit 23 (partial transfer, permission denied) and 24 (vanished files)
are expected in non-root backups. These no longer trigger retries or
fail the backup — they log a warning and continue.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
shuki
2026-03-06 16:51:13 +02:00
parent 3e4b906f7e
commit 18af43936c
1 changed files with 13 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
lib/transfer.sh
View File

@@ -57,6 +57,14 @@ rsync_to_remote() {
return 0 return 0
fi fi
# Exit 23 = partial transfer (permission denied on some files)
# Exit 24 = vanished source files (deleted during transfer)
# Both are expected in non-root backups — treat as success with warning
if (( rc == 23 || rc == 24 )); then
log_warn "rsync completed with warnings (exit $rc): some files could not be transferred"
return 0
fi
log_warn "rsync failed (exit $rc), attempt $attempt/$max_retries" log_warn "rsync failed (exit $rc), attempt $attempt/$max_retries"
if (( attempt < max_retries )); then if (( attempt < max_retries )); then
@@ -116,6 +124,11 @@ rsync_local() {
return 0 return 0
fi fi
if (( rc == 23 || rc == 24 )); then
log_warn "rsync (local) completed with warnings (exit $rc): some files could not be transferred"
return 0
fi
log_warn "rsync (local) failed (exit $rc), attempt $attempt/$max_retries" log_warn "rsync (local) failed (exit $rc), attempt $attempt/$max_retries"
if (( attempt < max_retries )); then if (( attempt < max_retries )); then