Skip to content

[cherry-pick] Fix image pull after a failure#1918

Merged
mlaventure merged 1 commit intocontainerd:release/1.0from
crosbymichael:cherry-pull-fix
Dec 15, 2017
Merged

[cherry-pick] Fix image pull after a failure#1918
mlaventure merged 1 commit intocontainerd:release/1.0from
crosbymichael:cherry-pull-fix

Conversation

@crosbymichael
Copy link
Copy Markdown
Member

When resuming from a failed pull writer.Truncate() was not
seeking to the proper position in the file. This caused writes to
happen after the previously written content, instead of at the start
of the file.

Signed-off-by: Daniel Nephin [email protected]

When resuming from a failed pull writer.Truncate() was not
seeking to the proper position in the file. This caused writes to
happen after the previously written content, instead of at the start
of the file.

Signed-off-by: Daniel Nephin <[email protected]>
@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #1918 into release/1.0 will increase coverage by 0.04%.
The diff coverage is 0%.

Impacted file tree graph

@@              Coverage Diff               @@
##           release/1.0   #1918      +/-   ##
==============================================
+ Coverage        50.66%   50.7%   +0.04%     
==============================================
  Files               81      81              
  Lines             7159    7161       +2     
==============================================
+ Hits              3627    3631       +4     
+ Misses            2849    2845       -4     
- Partials           683     685       +2
Flag Coverage Δ
#linux 50.7% <0%> (+0.04%) ⬆️
Impacted Files Coverage Δ
content/helpers.go 37.93% <0%> (ø) ⬆️
content/local/writer.go 60% <0%> (+3.83%) ⬆️

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 dba5fa0...63e1963. Read the comment docs.

Copy link
Copy Markdown
Member

@estesp estesp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Random-Liu
Copy link
Copy Markdown
Member

Merge?

Copy link
Copy Markdown
Contributor

@mlaventure mlaventure left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

ref #1916

@mlaventure mlaventure added this to the 1.0.1 milestone Dec 15, 2017
@mlaventure mlaventure merged commit 6c7abf7 into containerd:release/1.0 Dec 15, 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.

6 participants