Skip to content

[release/1.3] snapshots: return error if readSnapshot fails#3822

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

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

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]>
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Nov 12, 2019

Build succeeded.

@codecov-io
Copy link
Copy Markdown

codecov-io commented Nov 12, 2019

Codecov Report

Merging #3822 into release/1.3 will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@             Coverage Diff              @@
##           release/1.3    #3822   +/-   ##
============================================
  Coverage        45.59%   45.59%           
============================================
  Files              116      116           
  Lines            11465    11465           
============================================
  Hits              5227     5227           
  Misses            5338     5338           
  Partials           900      900
Flag Coverage Δ
#linux 45.59% <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 780b5d5...e19c621. Read the comment docs.

@crosbymichael
Copy link
Copy Markdown
Member

LGTM

@crosbymichael crosbymichael merged commit b5ede20 into containerd:release/1.3 Nov 12, 2019
@fuweid fuweid deleted the cherry-pick-3819-1.3 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