Skip to content

[release/1.2] snapshots: return error if readSnapshot fails#3821

Merged
crosbymichael merged 1 commit intocontainerd:release/1.2from
fuweid:cherry-pick-3819-1.2
Nov 12, 2019
Merged

[release/1.2] snapshots: return error if readSnapshot fails#3821
crosbymichael merged 1 commit intocontainerd:release/1.2from
fuweid:cherry-pick-3819-1.2

Conversation

@fuweid
Copy link
Copy Markdown
Member

@fuweid fuweid commented Nov 12, 2019

Signed-off-by: Wei Fu [email protected]
(cherry picked from commit 843423e)
Signed-off-by: Wei Fu [email protected]

from #3819

Signed-off-by: Wei Fu <[email protected]>
(cherry picked from commit 843423e)
Signed-off-by: Wei Fu <[email protected]>
@fuweid fuweid changed the title snapshots: return error if readSnapshot fails [release/1.2] snapshots: return error if readSnapshot fails Nov 12, 2019
@codecov-io
Copy link
Copy Markdown

codecov-io commented Nov 12, 2019

Codecov Report

Merging #3821 into release/1.2 will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@             Coverage Diff              @@
##           release/1.2    #3821   +/-   ##
============================================
  Coverage        47.87%   47.87%           
============================================
  Files               91       91           
  Lines             8483     8483           
============================================
  Hits              4061     4061           
  Misses            3688     3688           
  Partials           734      734
Flag Coverage Δ
#linux 47.87% <0%> (ø) ⬆️
Impacted Files Coverage Δ
snapshots/storage/bolt.go 63.6% <0%> (ø) ⬆️

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 a287c08...128664b. Read the comment docs.

@crosbymichael
Copy link
Copy Markdown
Member

LGTM

@crosbymichael crosbymichael merged commit db4537e into containerd:release/1.2 Nov 12, 2019
@fuweid fuweid deleted the cherry-pick-3819-1.2 branch April 22, 2020 11:34
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.

4 participants