feat: allow an optional platform definition for container arch#33
feat: allow an optional platform definition for container arch#33chickenandpork wants to merge 1 commit intosalesforce:mainfrom
Conversation
|
Thanks for the contribution! Before we can merge this, we need @chickenandpork to sign the Salesforce Inc. Contributor License Agreement. |
e3f9104 to
8674094
Compare
|
Thanks for the contribution! Unfortunately we can't verify the commit author(s): Allan Clark <a***@c***.com>. One possible solution is to add that email to your GitHub account. Alternatively you can change your commits to another email and force push the change. After getting your commits associated with your GitHub account, sign the Salesforce Inc. Contributor License Agreement and this Pull Request will be revalidated. |
8674094 to
582a746
Compare
|
(@kriscfoster waiting on the CLA-bot to catch up... -- iterated emails to find one on my account) |
582a746 to
fcbcb21
Compare
|
(pushed a different email address to get the CLA workflow to restart) |
|
@kriscfoster I see I don't see this failure, but I'm on a darwin/arm64, maybe for some reason the platform is preferring am arm64, so it doesn't hurt me but hurts th x86_64/amd64 arch ? I'm guessing a bit :( Maybe we build a multi-arch-manifest image, with arch matching the container, and let the host docker decide which architecture fits? (such would be another |
Yeah, that sounds possible. I'm not so familiar with this issue so unfortunately don't have a quick answer :( |
This enhancement (with accompanying example) shows how to set a list of platforms for the os/arch of the docker env so that the go_test can be executed in a controlled platform. For example, this allows a macOS/arm64 build to trigger a linux/amd64 test container.
The example is duplicated and extended from the existing
go-test-image-testto preserve that example, and to show what changes when the pathname of the test changes (which some additional documentation).I was unable, however, to create a functional example with a second built image in the docker-compose: the functionality of
local_image_targetsanddataattributes just didn't cause the install script from theoci_load()to be present, earningno install script present.fixes #32