From d5a1979d879dd674629bcd164eef6c97da9fa727 Mon Sep 17 00:00:00 2001 From: Thomas Waldmann Date: Mon, 4 Nov 2019 18:28:50 +0100 Subject: [PATCH] commit-time free space calc: ignore bad compact map entries, fixes #4796 at least it does not crash now when committing. the question why the compact map points to a missing segment file is not answered yet, there might be another problem... --- src/borg/repository.py | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/src/borg/repository.py b/src/borg/repository.py index 5f78c8b61..6196e1a67 100644 --- a/src/borg/repository.py +++ b/src/borg/repository.py @@ -669,7 +669,13 @@ def check_free_space(self): if len(self.compact) < 10: # This is mostly for the test suite to avoid overestimated free space needs. This can be annoying # if TMP is a small-ish tmpfs. - compact_working_space = sum(self.io.segment_size(segment) - free for segment, free in self.compact.items()) + compact_working_space = 0 + for segment, free in self.compact.items(): + try: + compact_working_space += self.io.segment_size(segment) - free + except FileNotFoundError: + # looks like self.compact is referring to a non-existent segment file, ignore it. + pass logger.debug('check_free_space: few segments, not requiring a full free segment') compact_working_space = min(compact_working_space, full_segment_size) logger.debug('check_free_space: calculated working space for compact as %d bytes', compact_working_space)