Skip to content

Add imageDigests to output to support multi-arch builds on native runners #373

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

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

jasondamour
Copy link

@jasondamour jasondamour commented May 19, 2025

  • Added imageDigests output to action
  • Added docs on how to utilize imageDigests to do multi-arch builds on native runners in Github Actions

@jasondamour jasondamour requested review from stuartleeks and a team as code owners May 19, 2025 17:57
@jasondamour
Copy link
Author

@microsoft-github-policy-service agree

@jasondamour
Copy link
Author

CC @stuartleeks

@rjstone
Copy link

rjstone commented Jun 23, 2025

This is also needed for actions/attest-build-provenance

@jasondamour
Copy link
Author

Ok I'm running into a little issue. If platform is specified as an input, then the image is output as type=oci,dest=/tmp/output.tar, and can't be found by docker inspect to get the digest.

Would anything break if I moved loading the tar from the runPost to the main script? Will try it out, but warn me of potential pitfalls pls

@jasondamour
Copy link
Author

Ok, this is ready for review again. I have a sample workflow successfully working end-to-end (unfortunately in a private repo). The key change I made was coping the oci tar to docker at the end of the build, in addition to the beginning of the post-run step. I need extra review on whether thats ok, and whether I can safely remove the copy from the start of the post-run step.

@jasondamour jasondamour requested a review from mfcollins3 July 7, 2025 17:55
@jasondamour
Copy link
Author

@chrmarti Can you help this?

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.

3 participants