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

fix(merge-oci-digests): Show --all packages #8

Merged

Conversation

nderjung
Copy link
Member

@nderjung nderjung commented Nov 1, 2023

Following the merge of kraftkit#945, without specifying the --all flag, the host architecture and platform are detected and used to reduce the output. In this CI/CD context, we wish to show all variants which requires the use of this flag. Without this flag, the GitHub Actions workflow fails as it cannot determine the host platform.

Following the merge of kraftkit#945, without specifying the `--all`
flag, the host architecture and platform are detected and used to reduce
the output.  In this CI/CD context, we wish to show all variants which
requires the use of this flag.  Without this flag, the GitHub Actions
workflow fails as it cannot determine the host platform.

Signed-off-by: Alexander Jung <alex@unikraft.io>
@nderjung nderjung requested a review from razvand November 2, 2023 08:52
Copy link
Contributor

@razvand razvand left a comment

Choose a reason for hiding this comment

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

Thanks, @nderjung.

Reviewed-by: Razvan Deaconescu razvand@unikraft.io
Approved-by: Razvan Deaconescu razvand@unikraft.io

@razvand razvand merged commit 46c2059 into main Nov 2, 2023
@razvand razvand deleted the nderjung/fix/github/actions/merge-oci-digests/show-all branch November 2, 2023 14:02
razvand pushed a commit to unikraft-upb/catalog that referenced this pull request Dec 19, 2023
feat(utils): Introduce bincompat generator script

Reviewed-by: Mihnea Firoiu <mihneafiroiu0@gmail.com>
Approved-by: Stefan Jumarea <stefanjumarea02@gmail.com>
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

2 participants