Skip to content

Conversation

fd0
Copy link
Member

@fd0 fd0 commented Jul 18, 2017

This fixes a bug introduced in c79fb6f where the index file after a prune contains pack files that do not exist any more. Restic will detect this during backup and abort with an error message until the user runs prune or rebuild-index again.

fd0 added 2 commits July 18, 2017 23:07
This fixes a bug introduced in c79fb6f
where the index file after a prune contains pack files that do not exist
any more. Restic will detect this during backup and abort with an error
message until the user runs `prune` or `rebuild-index` again.
@fd0 fd0 mentioned this pull request Jul 19, 2017
11 tasks
@codecov-io
Copy link

Codecov Report

Merging #1115 into master will increase coverage by 0.08%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1115      +/-   ##
==========================================
+ Coverage    52.9%   52.99%   +0.08%     
==========================================
  Files         118      118              
  Lines       11299    11300       +1     
==========================================
+ Hits         5978     5988      +10     
+ Misses       4610     4602       -8     
+ Partials      711      710       -1
Impacted Files Coverage Δ
src/cmds/restic/cmd_prune.go 63.26% <100%> (+2.75%) ⬆️
src/restic/backend/s3/s3.go 61.03% <0%> (+1.37%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6c05353...bd31281. Read the comment docs.

@fd0 fd0 merged commit bd31281 into master Jul 19, 2017
fd0 added a commit that referenced this pull request Jul 19, 2017
prune: Fix newly created index
@fd0 fd0 deleted the fix-prune-index branch July 19, 2017 15:12
@gvangool gvangool mentioned this pull request Jul 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants