Skip to content

Commit 9aaf044

Browse files
committed
Merge branch 'main' into feat/better-modelling-list
2 parents da9f232 + 268465b commit 9aaf044

1 file changed

Lines changed: 16 additions & 5 deletions

File tree

.github/workflows/deploy-cloud.yaml

Lines changed: 16 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,6 @@ jobs:
4242
SENTRY_PROJECT: dashboard
4343
SENTRY_URL_PREFIX: "~/dashboard/static"
4444
ENVIRONMENT: ${{ github.event.client_payload.deployment_name }}
45-
REGION: ${{ github.event.client_payload.region }}
4645
SENTRY_DSN: ${{ secrets.SENTRY_DSN }}
4746
SENTRY_AUTH_TOKEN: ${{ secrets.SENTRY_AUTH_TOKEN }}
4847
EXTENSIONS_API_URL: "https://apps.saleor.io/api/v1/extensions"
@@ -53,9 +52,6 @@ jobs:
5352
ENABLED_SERVICE_NAME_HEADER: true
5453
ONBOARDING_USER_JOINED_DATE_THRESHOLD: ${{ vars.CLOUD_ONBOARDING_USER_JOINED_DATE_THRESHOLD }}
5554
steps:
56-
- name: Check region
57-
if: ${{ !contains(fromJSON('["eu", "us"]'), env.REGION) }}
58-
run: echo 'Unknown region provided' >&2; exit 1
5955

6056
- uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
6157
with:
@@ -97,14 +93,23 @@ jobs:
9793

9894
deploy:
9995
runs-on: ubuntu-24.04
96+
needs: [build]
97+
10098
environment:
10199
deployment: false
102100
name: deploy-cloud
103101

104102
permissions:
105103
id-token: write
106104

105+
env:
106+
REGION: ${{ github.event.client_payload.region }}
107+
107108
steps:
109+
- name: Check region
110+
if: ${{ !contains(fromJSON('["eu", "us"]'), env.REGION) }}
111+
run: echo 'Unknown region provided' >&2; exit 1
112+
108113
- uses: actions/download-artifact@70fc10c6e5e1ce46ad2ea6f2b72d43f7d47b13c3 # v8.0.0
109114
with:
110115
name: build
@@ -129,7 +134,10 @@ jobs:
129134
env:
130135
DEPLOYMENT_BUCKET: ${{ secrets.AWS_CLOUD_EU_DEPLOYMENT_BUCKET }}
131136
CDN_DISTRIBUTIONS: ${{ secrets.AWS_CLOUD_EU_CDN_DISTRIBUTIONS }}
137+
ENVIRONMENT: ${{ needs.build.outputs.ENVIRONMENT }}
132138
run: |
139+
set -eu
140+
133141
aws s3 sync build/dashboard "s3://${DEPLOYMENT_BUCKET}/${ENVIRONMENT}/static/"
134142
aws s3 cp build/dashboard/index.html "s3://${DEPLOYMENT_BUCKET}/${ENVIRONMENT}/"
135143
@@ -144,7 +152,10 @@ jobs:
144152
env:
145153
DEPLOYMENT_BUCKET: ${{ secrets.AWS_CLOUD_US_DEPLOYMENT_BUCKET }}
146154
CDN_DISTRIBUTIONS: ${{ secrets.AWS_CLOUD_US_CDN_DISTRIBUTIONS }}
155+
ENVIRONMENT: ${{ needs.build.outputs.ENVIRONMENT }}
147156
run: |
157+
set -eu
158+
148159
aws s3 sync build/dashboard "s3://${DEPLOYMENT_BUCKET}/${ENVIRONMENT}/static/"
149160
aws s3 cp build/dashboard/index.html "s3://${DEPLOYMENT_BUCKET}/${ENVIRONMENT}/"
150161
@@ -155,7 +166,7 @@ jobs:
155166
done
156167
157168
notify-slack:
158-
needs: build
169+
needs: [build, deploy]
159170
if: always()
160171
permissions: {}
161172
uses: saleor/saleor-internal-actions/.github/workflows/notify-slack.yaml@eb0c692da7bf13f5e1a82c17488b24c514dd10a1 # v1.10.0

0 commit comments

Comments
 (0)