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 a --artifact-annotation for --artifact #24788

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nalind
Copy link
Member

@nalind nalind commented Dec 6, 2024

When adding an artifact to a manifest list / image index with manifest add --artifact provide a --artifact-annotation flag which can be used to set annotations in the artifact manifest that we're generating.

Does this PR introduce a user-facing change?

When adding files as artifacts to image indexes using `podman manifest add --artifact`, the `--artifact-annotation` flag can now be used to add annotations to the artifact manifest that will be generated.

@openshift-ci openshift-ci bot added release-note do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. labels Dec 6, 2024
Copy link
Contributor

openshift-ci bot commented Dec 6, 2024

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: nalind
Once this PR has been reviewed and has the lgtm label, please assign mheon for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found 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

@nalind nalind force-pushed the manifest-add-artifact-annotation branch 3 times, most recently from 1b4bd4c to 1bb9095 Compare December 9, 2024 15:17
When adding an artifact to a manifest list / image index with `manifest
add --artifact` provide a `--artifact-annotation` flag which can be used
to set annotations in the artifact manifest that we're generating.

Fix a client-side hang when using --artifact.

Signed-off-by: Nalin Dahyabhai <[email protected]>
@nalind nalind force-pushed the manifest-add-artifact-annotation branch from 1bb9095 to bd347db Compare December 9, 2024 18:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. release-note
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant