Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use a single custom annotation for export #3296

Merged
merged 1 commit into from May 22, 2019

Conversation

dmcgowan
Copy link
Member

Remove annotation prefix and add multiple index records
for manifests with multiple image names. This makes the
custom annotation more consistent with the OCI ref name
annotation. Additionally, ensure the OCI image annotation
always represents the tag (partial image name) as recommended
by the specification. The containerd image name annotation
will always contain the full image name.

@dmcgowan dmcgowan requested a review from AkihiroSuda May 21, 2019 23:48
@theopenlab-ci
Copy link

theopenlab-ci bot commented May 21, 2019

Build succeeded.

@dmcgowan dmcgowan force-pushed the fix-export-labels branch 2 times, most recently from dcc2169 to 5e2d7ef Compare May 22, 2019 00:21
Remove annotation prefix and add multiple index records
for manifests with multiple image names. This makes the
custom annotation more consistent with the OCI image
annotation. Additionally, ensure the OCI image annotation
always represents the tag (partial image name) as recommended
by the specification. The containerd image name annotation
will always contain the full image name.

Signed-off-by: Derek McGowan <derek@mcgstyle.net>
@theopenlab-ci
Copy link

theopenlab-ci bot commented May 22, 2019

Build succeeded.

@codecov-io
Copy link

codecov-io commented May 22, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3296   +/-   ##
=======================================
  Coverage   44.62%   44.62%           
=======================================
  Files         112      112           
  Lines       12174    12174           
=======================================
  Hits         5433     5433           
  Misses       5907     5907           
  Partials      834      834
Flag Coverage Δ
#linux 48.52% <ø> (ø) ⬆️
#windows 39.89% <ø> (ø) ⬆️

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 2088fc9...5e2d7ef. Read the comment docs.

@crosbymichael
Copy link
Member

LGTM

@crosbymichael crosbymichael merged commit b7f093e into containerd:master May 22, 2019
@dmcgowan dmcgowan deleted the fix-export-labels 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.

None yet

4 participants