Skip to content

Commit 5e4ec90

Browse files
authored
Merge pull request #654 from Dstack-TEE/fix/docker-build-check-fork-prs
ci: fix docker-build-check for PRs from forks
2 parents 282eeb2 + 4d2d9da commit 5e4ec90

1 file changed

Lines changed: 18 additions & 14 deletions

File tree

.github/workflows/docker-build-check.yml

Lines changed: 18 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,10 @@ on:
1010
pull_request:
1111
branches: [ master, next, dev-* ]
1212

13+
env:
14+
DSTACK_REV: ${{ github.event.pull_request.head.sha || github.sha }}
15+
DSTACK_SRC_URL: ${{ github.event.pull_request.head.repo.clone_url || format('{0}/{1}', github.server_url, github.repository) }}
16+
1317
jobs:
1418
gateway:
1519
runs-on: ubuntu-latest
@@ -30,8 +34,8 @@ jobs:
3034
build-contexts: |
3135
build-shared=build/shared
3236
build-args: |
33-
DSTACK_REV=${{ github.event.pull_request.head.sha || github.sha }}
34-
DSTACK_SRC_URL=${{ github.server_url }}/${{ github.repository }}
37+
DSTACK_REV=${{ env.DSTACK_REV }}
38+
DSTACK_SRC_URL=${{ env.DSTACK_SRC_URL }}
3539
3640
- name: Verify pinned packages
3741
run: |
@@ -46,8 +50,8 @@ jobs:
4650
--tag gateway-builder-check:latest \
4751
--provenance=false \
4852
--build-context build-shared=build/shared \
49-
--build-arg "DSTACK_REV=${{ github.event.pull_request.head.sha || github.sha }}" \
50-
--build-arg "DSTACK_SRC_URL=${{ github.server_url }}/${{ github.repository }}" \
53+
--build-arg "DSTACK_REV=${DSTACK_REV}" \
54+
--build-arg "DSTACK_SRC_URL=${DSTACK_SRC_URL}" \
5155
gateway/dstack-app/builder
5256
5357
- name: Verify builder pinned packages
@@ -74,8 +78,8 @@ jobs:
7478
build-contexts: |
7579
build-shared=build/shared
7680
build-args: |
77-
DSTACK_REV=${{ github.event.pull_request.head.sha || github.sha }}
78-
DSTACK_SRC_URL=${{ github.server_url }}/${{ github.repository }}
81+
DSTACK_REV=${{ env.DSTACK_REV }}
82+
DSTACK_SRC_URL=${{ env.DSTACK_SRC_URL }}
7983
8084
- name: Verify pinned packages (qemu stage)
8185
run: |
@@ -90,8 +94,8 @@ jobs:
9094
--tag kms-builder-check:latest \
9195
--provenance=false \
9296
--build-context build-shared=build/shared \
93-
--build-arg "DSTACK_REV=${{ github.event.pull_request.head.sha || github.sha }}" \
94-
--build-arg "DSTACK_SRC_URL=${{ github.server_url }}/${{ github.repository }}" \
97+
--build-arg "DSTACK_REV=${DSTACK_REV}" \
98+
--build-arg "DSTACK_SRC_URL=${DSTACK_SRC_URL}" \
9599
kms/dstack-app/builder
96100
97101
- name: Verify builder pinned packages
@@ -125,8 +129,8 @@ jobs:
125129
build-contexts: |
126130
build-shared=build/shared
127131
build-args: |
128-
DSTACK_REV=${{ github.event.pull_request.head.sha || github.sha }}
129-
DSTACK_SRC_URL=${{ github.server_url }}/${{ github.repository }}
132+
DSTACK_REV=${{ env.DSTACK_REV }}
133+
DSTACK_SRC_URL=${{ env.DSTACK_SRC_URL }}
130134
131135
- name: Verify pinned packages (runtime)
132136
run: |
@@ -142,8 +146,8 @@ jobs:
142146
--provenance=false \
143147
--file verifier/builder/Dockerfile \
144148
--build-context build-shared=build/shared \
145-
--build-arg "DSTACK_REV=${{ github.event.pull_request.head.sha || github.sha }}" \
146-
--build-arg "DSTACK_SRC_URL=${{ github.server_url }}/${{ github.repository }}" \
149+
--build-arg "DSTACK_REV=${DSTACK_REV}" \
150+
--build-arg "DSTACK_SRC_URL=${DSTACK_SRC_URL}" \
147151
verifier
148152
149153
- name: Verify builder pinned packages
@@ -160,8 +164,8 @@ jobs:
160164
--provenance=false \
161165
--file verifier/builder/Dockerfile \
162166
--build-context build-shared=build/shared \
163-
--build-arg "DSTACK_REV=${{ github.event.pull_request.head.sha || github.sha }}" \
164-
--build-arg "DSTACK_SRC_URL=${{ github.server_url }}/${{ github.repository }}" \
167+
--build-arg "DSTACK_REV=${DSTACK_REV}" \
168+
--build-arg "DSTACK_SRC_URL=${DSTACK_SRC_URL}" \
165169
verifier
166170
167171
- name: Verify qemu pinned packages

0 commit comments

Comments
 (0)