]> git.proxmox.com Git - proxmox-backup.git/commit
garbage_collection: only ignore 'missing chunk' errors
authorDominik Csapak <d.csapak@proxmox.com>
Mon, 1 Feb 2021 08:07:12 +0000 (09:07 +0100)
committerDietmar Maurer <dietmar@proxmox.com>
Mon, 1 Feb 2021 08:18:59 +0000 (09:18 +0100)
commit1399c592d1032b2dbc2fc2c1e967561046ddb5e0
tree3f98832388be9b0ad09640d80df595d4bdb09ea6
parent9883b54cbad440c088f2af97f112e3df7a33e1dc
garbage_collection: only ignore 'missing chunk' errors

with the fix for #2909 (improving handling missing chunks), we
changed from bailing to warning during a garbage collection when
updating the atime of a chunk.

but, updating the atime can not only fail when the chunk is missing,
but also on other occasions, e.g. no permissions or more importantly,
no space left on the device. in that case, the atime of a valid and used
chunk cannot be updated, and the second sweep of the gc will remove that chunk.
[0] is a real world example of that happening.

instead, only warn on really missin chunks, and bail on all other
errors.

0: https://forum.proxmox.com/threads/pbs-server-full-two-days-later-almost-empty.83274/

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
src/backup/datastore.rs