Skip to content

Commit 639cf43

Browse files
committed
Update
[ghstack-poisoned]
2 parents 6766019 + 8b0d362 commit 639cf43

1 file changed

Lines changed: 14 additions & 1 deletion

File tree

.github/workflows/gha-log-uploader-lambda.yml

Lines changed: 14 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,8 +25,14 @@ jobs:
2525
with:
2626
python-version: '3.12'
2727
cache: pip
28-
- run: pip3 install -r requirements.txt pytest
28+
- run: pip3 install -r requirements.txt pytest==7.4.0
2929
- run: pytest -v test_lambda_function.py
30+
# Build the deployment package too. `prepare` resolves manylinux wheels for
31+
# the lambda's platform rather than the runner's, and `verify` refuses a zip
32+
# that is missing a vendored module -- neither of which the plain install
33+
# above exercises. Without this a bad requirements.txt only fails at deploy
34+
# time, which is to say on main.
35+
- run: make prepare
3036

3137
deploy:
3238
needs: test
@@ -35,6 +41,13 @@ jobs:
3541
permissions:
3642
id-token: write
3743
contents: read
44+
# Serialise deploys: two merges in quick succession would otherwise race on
45+
# update-function-code, and if the older run finishes last the function is
46+
# left on the older commit. Queue rather than cancel -- cancelling a deploy
47+
# mid-flight is worse than waiting for it.
48+
concurrency:
49+
group: ${{ github.workflow }}-deploy
50+
cancel-in-progress: false
3851
steps:
3952
- name: configure aws credentials
4053
uses: aws-actions/configure-aws-credentials@67fbcbb121271f7775d2e7715933280b06314838 # v1.7.0

0 commit comments

Comments
 (0)