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 oci image index script #637

Closed
wants to merge 1 commit into from

Conversation

tobiaskohlbau
Copy link

@tobiaskohlbau tobiaskohlbau commented Jun 24, 2024

The script does not work for multiarch images as it does not append to the manifests. Instead it replaces with the latest one which results in the last architecture to be the only one supported. JQ has an option to append an element to an array.

Fixes #638

The script does not work for multiarch images as it does not append
to the manifests. Instead it replaces with the latest one which results
in the last architecture to be the only one supported. JQ has an option
to append an element to an array.

Signed-off-by: Tobias Kohlbau <[email protected]>
@tobiaskohlbau
Copy link
Author

tobiaskohlbau commented Jun 24, 2024

We at @myopenfactory deployed these changes into our monorepo and confirmed that it works like the 1.x versions of rules_oci.

@seh
Copy link
Contributor

seh commented Jun 24, 2024

This is a duplicate of #633.

@tobiaskohlbau
Copy link
Author

tobiaskohlbau commented Jun 25, 2024

This is a duplicate of #633.

Oh my bad, did look over the PRs but did not associate this. Let's try to get the other one in. Maybe you could reference the ticket #638 to solve by your ticket.

Edit:
Nevermind you already referenced it.

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.

2 participants