Skip to content

set base-image annotations#3423

Merged
openshift-ci[bot] merged 1 commit intocontainers:mainfrom
vrothberg:fix-3415
Aug 3, 2021
Merged

set base-image annotations#3423
openshift-ci[bot] merged 1 commit intocontainers:mainfrom
vrothberg:fix-3415

Conversation

@vrothberg
Copy link
Copy Markdown
Member

Since opencontainers/image-spec/pull/822/ the OCI spec supports two new
annotations to set the fully-qualified name and the digest of the base
image.

Signed-off-by: Valentin Rothberg rothberg@redhat.com

@rhatdan @nalind @TomSweeneyRedHat @imjasonh PTAL

@openshift-ci openshift-ci bot added the approved label Aug 3, 2021
Copy link
Copy Markdown

@imjasonh imjasonh left a comment

Choose a reason for hiding this comment

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

Woohoo! Thanks for implementing this so quickly!

run_buildah images -a
expect_line_count 4

# Note that the base-image annotations are empty here since a Container with
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Nice. 👍

Since opencontainers/image-spec/pull/822/ the OCI spec supports two new
annotations to set the fully-qualified name and the digest of the base
image.

Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
@vrothberg
Copy link
Copy Markdown
Member Author

Had to massage some other integration tests which barked with the new annotations.

Copy link
Copy Markdown
Member

@rhatdan rhatdan left a comment

Choose a reason for hiding this comment

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

LGTM

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci bot commented Aug 3, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: imjasonh, rhatdan, vrothberg

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@rhatdan
Copy link
Copy Markdown
Member

rhatdan commented Aug 3, 2021

@TomSweeneyRedHat
Copy link
Copy Markdown
Member

LGTM

@rhatdan
Copy link
Copy Markdown
Member

rhatdan commented Aug 3, 2021

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Aug 3, 2021
@openshift-ci openshift-ci bot merged commit 0022921 into containers:main Aug 3, 2021
@vrothberg vrothberg deleted the fix-3415 branch August 4, 2021 06:27
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants