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

manifest add: add --artifact-annotation #5854

Merged

Conversation

nalind
Copy link
Member

@nalind nalind commented Dec 2, 2024

What type of PR is this?

/kind feature

What this PR does / why we need it:

Add a --artifact-annotation flag to buildah manifest add that can be used to set annotations in the artifact manifest that we generate and then add to an image index.

How to verify it

Updated integration test!

Which issue(s) this PR fixes:

None

Special notes for your reviewer:

Does this PR introduce a user-facing change?

`buildah manifest add` now includes a `--artifact-annotation` flag which can be used to add an annotation to the artifact manifest which is generated and then added to the image index.

@openshift-ci openshift-ci bot added the kind/feature Categorizes issue or PR as related to a new feature. label Dec 2, 2024
Copy link
Contributor

openshift-ci bot commented Dec 2, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: nalind

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

The pull request process is described here

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

@openshift-ci openshift-ci bot added the approved label Dec 2, 2024
Copy link

Ephemeral COPR build failed. @containers/packit-build please check.

3 similar comments
Copy link

Ephemeral COPR build failed. @containers/packit-build please check.

Copy link

Ephemeral COPR build failed. @containers/packit-build please check.

Copy link

Ephemeral COPR build failed. @containers/packit-build please check.

@nalind nalind force-pushed the manifest-add-artifact-annotation branch 2 times, most recently from d8c7511 to a1d22fb Compare December 6, 2024 18:39
@TomSweeneyRedHat
Copy link
Member

LGTM
but needs a rebase.

Add a --artifact-annotation flag to `buildah manifest add` that can be
used to set annotations in the artifact manifest that we generate and
then add to an image index.

Signed-off-by: Nalin Dahyabhai <[email protected]>
@nalind nalind force-pushed the manifest-add-artifact-annotation branch from a1d22fb to 5f39753 Compare December 12, 2024 19:09
@rhatdan
Copy link
Member

rhatdan commented Dec 13, 2024

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Dec 13, 2024
@openshift-merge-bot openshift-merge-bot bot merged commit 81015fa into containers:main Dec 13, 2024
19 checks passed
@nalind nalind deleted the manifest-add-artifact-annotation branch December 13, 2024 20:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved kind/feature Categorizes issue or PR as related to a new feature. lgtm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants