Skip to content

Commit

Permalink
Merge pull request #8077 from ThomasWaldmann/index-check-value-errors…
Browse files Browse the repository at this point in the history
…-1.4

check: fix return code for index entry value discrepancies
  • Loading branch information
ThomasWaldmann committed Feb 8, 2024
2 parents 6292db2 + 97fe48c commit 92669ab
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions src/borg/repository.py
Original file line number Diff line number Diff line change
Expand Up @@ -1015,10 +1015,10 @@ def check(self, repair=False, save_space=False, max_duration=0):
raise ValueError(self.path + " is in append-only mode")
error_found = False

def report_error(msg):
def report_error(msg, *args):
nonlocal error_found
error_found = True
logger.error(msg)
logger.error(msg, *args)

logger.info('Starting repository check')
assert not self._active_txn
Expand Down Expand Up @@ -1105,22 +1105,22 @@ def report_error(msg):
# self.index = "as rebuilt in-memory from segments"
if len(current_index) != len(self.index):
report_error('Index object count mismatch.')
logger.error('committed index: %d objects', len(current_index))
logger.error('rebuilt index: %d objects', len(self.index))
report_error('committed index: %d objects', len(current_index))
report_error('rebuilt index: %d objects', len(self.index))
else:
logger.info('Index object count match.')
line_format = 'ID: %-64s rebuilt index: %-16s committed index: %-16s'
not_found = '<not found>'
for key, value in self.index.iteritems():
current_value = current_index.get(key, not_found)
if current_value != value:
logger.warning(line_format, bin_to_hex(key), value, current_value)
report_error(line_format, bin_to_hex(key), value, current_value)
for key, current_value in current_index.iteritems():
if key in self.index:
continue
value = self.index.get(key, not_found)
if current_value != value:
logger.warning(line_format, bin_to_hex(key), value, current_value)
report_error(line_format, bin_to_hex(key), value, current_value)
if repair:
self.write_index()
self.rollback()
Expand Down

0 comments on commit 92669ab

Please sign in to comment.