Skip to content

Get media type key from context#3661

Merged
crosbymichael merged 1 commit intocontainerd:masterfrom
cpuguy83:custom_keys_for_fetch_hander
Sep 18, 2019
Merged

Get media type key from context#3661
crosbymichael merged 1 commit intocontainerd:masterfrom
cpuguy83:custom_keys_for_fetch_hander

Conversation

@cpuguy83
Copy link
Copy Markdown
Member

Adds a method for setting a custom media type key prefix used by the
fetch handler.
This allows both overwriting a built-in prefix (for reasons?) as well as
supplying a custom media type.

I added this because I was getting an error on FetchHandler when
pulling docker plugin images which have their own media type.

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Sep 18, 2019

Build succeeded.

@cpuguy83 cpuguy83 force-pushed the custom_keys_for_fetch_hander branch from 8b25d90 to a1d37f1 Compare September 18, 2019 17:50
@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Sep 18, 2019

Build succeeded.

Adds a method for setting a custom media type key prefix used by the
fetch handler.
This allows both overwriting a built-in prefix (for reasons?) as well as
supplying a custom media type.

I added this because I was getting an error on `FetchHandler` when
pulling docker plugin images which have their own media type.

Signed-off-by: Brian Goff <[email protected]>
@cpuguy83 cpuguy83 force-pushed the custom_keys_for_fetch_hander branch from a1d37f1 to bca374a Compare September 18, 2019 18:27
@codecov-io
Copy link
Copy Markdown

codecov-io commented Sep 18, 2019

Codecov Report

Merging #3661 into master will decrease coverage by 5.53%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3661      +/-   ##
==========================================
- Coverage   42.37%   36.83%   -5.54%     
==========================================
  Files         127       87      -40     
  Lines       14075    11943    -2132     
==========================================
- Hits         5964     4399    -1565     
+ Misses       7211     6930     -281     
+ Partials      900      614     -286
Flag Coverage Δ
#linux ?
#windows 36.83% <100%> (-0.44%) ⬇️
Impacted Files Coverage Δ
remotes/handlers.go 10.88% <100%> (ø)
archive/tar_opts.go 11.76% <0%> (-47.06%) ⬇️
cio/io.go 1.4% <0%> (-45.08%) ⬇️
snapshots/native/native.go 1.79% <0%> (-41.26%) ⬇️
archive/tar.go 18.95% <0%> (-29.24%) ⬇️
metadata/snapshot.go 23.86% <0%> (-24.05%) ⬇️
oci/spec_opts.go 28.93% <0%> (-2.3%) ⬇️
gc/scheduler/scheduler.go 66.34% <0%> (-0.97%) ⬇️
content/local/writer.go 57.69% <0%> (-0.97%) ⬇️
mount/temp_unix.go
... and 43 more

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 324a947...bca374a. Read the comment docs.

Copy link
Copy Markdown
Member

@dmcgowan dmcgowan left a comment

Choose a reason for hiding this comment

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

LGTM

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Sep 18, 2019

Build succeeded.

@dmcgowan dmcgowan added this to the 1.3 milestone Sep 18, 2019
@crosbymichael
Copy link
Copy Markdown
Member

LGTM

@crosbymichael crosbymichael merged commit a65aa5f into containerd:master Sep 18, 2019
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