Skip to content

Commit

Permalink
chore: fix dockerfile FROM statement (#1541)
Browse files Browse the repository at this point in the history
* fix dockerfile FROM statement

* add Dockerfile as a trigger for e2e tests

* add add Dockerfile.gaia as a trigger for tests

* Add FROM specifier to Dockerfile.gaia

* Use wildcard for git diff Dockerfile*

---------

Co-authored-by: insumity <[email protected]>
Co-authored-by: Philip Offtermatt <[email protected]>
  • Loading branch information
3 people authored Jan 9, 2024
1 parent fc4b9d5 commit be84275
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 2 deletions.
3 changes: 3 additions & 0 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -192,6 +192,7 @@ jobs:
**/go.sum
**/Makefile
Makefile
Dockerfile*
- name: e2e tests
if: env.GIT_DIFF
run: |
Expand Down Expand Up @@ -222,6 +223,7 @@ jobs:
**/go.sum
**/Makefile
Makefile
Dockerfile*
- name: cometmock tests
if: env.GIT_DIFF
run: |
Expand Down Expand Up @@ -252,6 +254,7 @@ jobs:
**/go.sum
**/Makefile
Makefile
Dockerfile*
- name: trace-e2e tests
if: env.GIT_DIFF
run: |
Expand Down
2 changes: 1 addition & 1 deletion Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ RUN go mod tidy
RUN make install

# Get Hermes build
FROM otacrew/hermes-ics:evidence-cmd AS hermes-builder
FROM --platform=linux/amd64 otacrew/hermes-ics:evidence-cmd AS hermes-builder

# Get CometMock
FROM ghcr.io/informalsystems/cometmock:v0.37.x as cometmock-builder
Expand Down
2 changes: 1 addition & 1 deletion Dockerfile.gaia
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ WORKDIR /interchain-security
RUN make install

# Get Hermes build
FROM ghcr.io/informalsystems/hermes:1.4.1 AS hermes-builder
FROM --platform=linux/amd64 ghcr.io/informalsystems/hermes:1.4.1 AS hermes-builder

FROM --platform=linux/amd64 fedora:36
RUN dnf update -y
Expand Down

0 comments on commit be84275

Please sign in to comment.