Skip to content

Commit

Permalink
Build with amd64 compatible image (#925)
Browse files Browse the repository at this point in the history
* Build with amd64 compatible image

* Update codeflow build spec

* Remove architecture attrs so that image is multiarch

* arm64 instead of amd

---------

Co-authored-by: David Roth <david.roth@coinbase.com>
  • Loading branch information
omkarb and dneilroth authored Aug 23, 2024
1 parent 680d980 commit f1f9126
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 12 deletions.
12 changes: 2 additions & 10 deletions .codeflow.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,16 +13,8 @@ build:
- BaldurECR:
name: docs
path: ./apps/base-docs/Dockerfile
- BaldurECR:
name: bridge
path: ./apps/bridge/Dockerfile
- BaldurECR:
name: goerli-bridge
path: ./apps/bridge/Dockerfile
- BaldurECR:
name: sepolia-bridge
path: ./apps/bridge/Dockerfile
multi_arch: true

operate:
slack_channels:
- "#base-codeflow-notifications"
- '#base-codeflow-notifications'
4 changes: 2 additions & 2 deletions tools/ci/build-dockerfile.sh
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,8 @@ set -eo pipefail

# We have to put this in a separate file as the pipeline
# command needs to be static, and this env var breaks that.
docker build --pull -f "$DOCKERFILE_PATH" -t ci_build_test_image --build-arg CODEFLOW_COMMIT_TAG .
docker build --platform=linux/arm64 --pull -f "$DOCKERFILE_PATH" -t ci_build_test_image --build-arg CODEFLOW_COMMIT_TAG .

# Cleanup after a successful build to avoid the agent
# running out of disk space!
docker image rm -f ci_build_test_image
docker image rm -f ci_build_test_image

0 comments on commit f1f9126

Please sign in to comment.