KurlesHS/restic
1
0
Fork 0
forked from TrueCloudLab/restic
Code Pull requests Activity

Don't retry when "Permission denied" occurs in local backend

Browse source
This commit is contained in:
tWido 2020-12-20 21:12:27 +01:00
parent f7c7c2f730
commit 7dab113035
2 changed files with 6 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
changelog/unreleased/issue-2453
View file

@ -5,8 +5,10 @@ restic retries the failing operation up to nine times (for a total of ten
attempts). It used to retry all backend operations, but now detects some attempts). It used to retry all backend operations, but now detects some
permanent error conditions so it can report fatal errors earlier. permanent error conditions so it can report fatal errors earlier.
Permanent failures include local disks being full and SSH connections Permanent failures include local disks being full, SSH connections
dropping. dropping and permission errors.
https://github.com/restic/restic/issues/2453 https://github.com/restic/restic/issues/2453
https://github.com/restic/restic/pull/3170 https://github.com/restic/restic/pull/3170
https://github.com/restic/restic/issues/3180
https://github.com/restic/restic/pull/3181

5
internal/backend/local/local.go
View file

@ -91,9 +91,8 @@ func (b *Local) Save(ctx context.Context, h restic.Handle, rd restic.RewindReade
filename := b.Filename(h) filename := b.Filename(h)
defer func() { defer func() {
// Mark non-retriable errors as such (currently only // Mark non-retriable errors as such
// "no space left on device"). if errors.Is(err, syscall.ENOSPC) || os.IsPermission(err) {
if errors.Is(err, syscall.ENOSPC) {
err = backoff.Permanent(err) err = backoff.Permanent(err)
} }
}() }()