Skip to content

Set uncompressed label on diff when already exists#2717

Merged
crosbymichael merged 1 commit intocontainerd:masterfrom
dmcgowan:diff-set-uncompressed-label
Oct 15, 2018
Merged

Set uncompressed label on diff when already exists#2717
crosbymichael merged 1 commit intocontainerd:masterfrom
dmcgowan:diff-set-uncompressed-label

Conversation

@dmcgowan
Copy link
Copy Markdown
Member

@dmcgowan dmcgowan commented Oct 15, 2018

Diff expects to set the uncompressed label on Commit, however if the blob already exists in the content store no labels will get set, requiring an Update. Check if the uncompressed label is on the blob and set it if not.

Fixes #2707

Diff expects to set the uncompressed label on Commit, however
if the blob already exists in the content store, no labels
will get set, requiring an Update. Check if the uncompressed
label is on the blob and set it if not.

Signed-off-by: Derek McGowan <[email protected]>
@dmcgowan dmcgowan added this to the 1.2 milestone Oct 15, 2018
@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #2717 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2717   +/-   ##
=======================================
  Coverage   43.54%   43.54%           
=======================================
  Files         101      101           
  Lines       10734    10734           
=======================================
  Hits         4674     4674           
  Misses       5326     5326           
  Partials      734      734
Flag Coverage Δ
#linux 47.2% <ø> (-0.03%) ⬇️
#windows 40.67% <ø> (ø) ⬆️

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 50c020f...bb197dd. 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

@crosbymichael
Copy link
Copy Markdown
Member

LGTM

@crosbymichael crosbymichael merged commit 3f5e6cd into containerd:master Oct 15, 2018
@dmcgowan dmcgowan deleted the diff-set-uncompressed-label branch September 10, 2019 17:43
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