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

chore: support ARM architecture #59

Merged
merged 12 commits into from
Mar 26, 2021
Merged

Conversation

SoManyHs
Copy link
Contributor

@SoManyHs SoManyHs commented Mar 12, 2021

Previously, the images pushed were only for amd64 architecture. This
adds support for building arm64 images. Their respective images will now
have a suffix indicating their architecture, e.g.
amazon/amazon-ecs-local-container-endpoints:latest-arm64

This change also ensures that go1.15 is being used to build.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Previously, the images pushed were only for amd64 architecture. This
adds support for building arm64 images. Their respective images will now
have a suffix indicating their architecture, e.g.
`amazon/amazon-ecs-local-container-endpoints:latest-arm64`

This change also ensures that go1.15 is being used to build.
Makefile Outdated Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
@SoManyHs SoManyHs marked this pull request as draft March 12, 2021 06:41
To ensure that the docker images being built are using the binary
compiled for the correct architecture, we are using the `--build-args`
flag to pass in the architecture as a parameter to the Dockerfile.
@SoManyHs SoManyHs force-pushed the arm-support branch 4 times, most recently from 811fff5 to a05597e Compare March 25, 2021 02:50
@SoManyHs SoManyHs marked this pull request as ready for review March 25, 2021 03:48
Use CDK to create a pipeline stack that uses a GitHub webhook and
CodeBuild to publish docker images for amd64 and arm64 architectures to
DockerHub.

TODO: Also publish to ECR Public
@SoManyHs
Copy link
Contributor Author

SoManyHs commented Mar 25, 2021

Tested new infrastructure by deploying pipeline to personal test account:

Screen Shot 2021-03-24 at 22 07 39

Images for both architectures were pushed to personal test DockerHub account, with images built for correct architecture:

Screen Shot 2021-03-25 at 16 19 50

@SoManyHs SoManyHs marked this pull request as ready for review March 25, 2021 23:19
Makefile Show resolved Hide resolved
@SoManyHs
Copy link
Contributor Author

With successful verification stage added:

Note: -- the verify target makes the calls and they are successful, but we may want a more robust way of testing the output of the curl commands.

Screen Shot 2021-03-25 at 22 23 56

Makefile Show resolved Hide resolved
infra/README.md Outdated Show resolved Hide resolved
infra/pipeline.ts Show resolved Hide resolved
infra/pipeline.ts Show resolved Hide resolved
infra/pipeline.ts Show resolved Hide resolved
infra/pipeline.ts Show resolved Hide resolved
@SoManyHs SoManyHs merged commit ea97409 into awslabs:mainline Mar 26, 2021
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.

4 participants