Skip to content

Commit 56b942c

Browse files
authored
Add tag push trigger to main workflow (#12829)
1 parent 8a45f31 commit 56b942c

File tree

3 files changed

+16
-14
lines changed

3 files changed

+16
-14
lines changed

.github/workflows/aws-main.yml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,8 @@ on:
1717
- '!docs/**'
1818
branches:
1919
- master
20+
tags:
21+
- 'v[0-9]+.[0-9]+.[0-9]+'
2022
pull_request:
2123
paths:
2224
- '**'
@@ -188,7 +190,7 @@ jobs:
188190
name: "Push images"
189191
runs-on: ubuntu-latest
190192
# push image on master, target branch not set, and the dependent steps were either successful or skipped
191-
if: github.ref == 'refs/heads/master' && !failure() && !cancelled() && github.repository == 'localstack/localstack'
193+
if: ( github.ref == 'refs/heads/master' || startsWith(github.ref, 'refs/tags/v') ) && !failure() && !cancelled() && github.repository == 'localstack/localstack'
192194
needs:
193195
# all tests need to be successful for the image to be pushed
194196
- test
@@ -267,7 +269,7 @@ jobs:
267269
268270
push-to-tinybird:
269271
name: Push Workflow Status to Tinybird
270-
if: always() && github.ref == 'refs/heads/master' && github.repository == 'localstack/localstack'
272+
if: always() && ( github.ref == 'refs/heads/master' || startsWith(github.ref, 'refs/tags/v') ) && github.repository == 'localstack/localstack'
271273
runs-on: ubuntu-latest
272274
needs:
273275
- test

.github/workflows/aws-tests-mamr.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ jobs:
6767

6868
push-to-tinybird:
6969
name: Push Workflow Status to Tinybird
70-
if: always() && github.ref == 'refs/heads/master' && github.repository == 'localstack/localstack'
70+
if: always() && ( github.ref == 'refs/heads/master' || startsWith(github.ref, 'refs/tags/v') ) && github.repository == 'localstack/localstack'
7171
runs-on: ubuntu-latest
7272
needs:
7373
- test-ma-mr

.github/workflows/aws-tests.yml

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -132,7 +132,7 @@ env:
132132
CI_COMMIT_SHA: ${{ github.sha }}
133133
CI_JOB_URL: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}/attempts/${{ github.run_attempt }}
134134
# report to tinybird if executed on master
135-
TINYBIRD_PYTEST_ARGS: "${{ github.repository == 'localstack/localstack' && github.ref == 'refs/heads/master' && '--report-to-tinybird ' || '' }}"
135+
TINYBIRD_PYTEST_ARGS: "${{ github.repository == 'localstack/localstack' && ( github.ref == 'refs/heads/master' || startsWith(github.ref, 'refs/tags/v') ) && '--report-to-tinybird ' || '' }}"
136136
DOCKER_PULL_SECRET_AVAILABLE: ${{ secrets.DOCKERHUB_PULL_USERNAME != '' && secrets.DOCKERHUB_PULL_TOKEN != '' && 'true' || 'false' }}
137137

138138

@@ -150,7 +150,7 @@ jobs:
150150
exclude:
151151
# skip the ARM integration tests in forks, and also if not on master/upgrade-dependencies and forceARMTests is not set to true
152152
# TODO ARM runners are not yet available for private repositories; skip them for potential private forks
153-
- runner: ${{ ((github.repository != 'localstack/localstack') || (github.ref != 'refs/heads/master' && github.ref != 'upgrade-dependencies' && inputs.forceARMTests == false)) && 'ubuntu-24.04-arm' || ''}}
153+
- runner: ${{ ((github.repository != 'localstack/localstack') || (github.ref != 'refs/heads/master' && !startsWith(github.ref, 'refs/tags/v') && github.ref != 'upgrade-dependencies' && inputs.forceARMTests == false)) && 'ubuntu-24.04-arm' || ''}}
154154
fail-fast: false
155155
runs-on: ${{ matrix.runner }}
156156
steps:
@@ -338,7 +338,7 @@ jobs:
338338
exclude:
339339
# skip the ARM integration tests in forks, and also if not on master/upgrade-dependencies and forceARMTests is not set to true
340340
# TODO ARM runners are not yet available for private repositories; skip them for potential private forks
341-
- runner: ${{ ((github.repository != 'localstack/localstack') || (github.ref != 'refs/heads/master' && github.ref != 'upgrade-dependencies' && inputs.forceARMTests == false)) && 'ubuntu-24.04-arm' || ''}}
341+
- runner: ${{ ((github.repository != 'localstack/localstack') || (github.ref != 'refs/heads/master' && !startsWith(github.ref, 'refs/tags/v') && github.ref != 'upgrade-dependencies' && inputs.forceARMTests == false)) && 'ubuntu-24.04-arm' || ''}}
342342
fail-fast: false
343343
runs-on: ${{ matrix.runner }}
344344
env:
@@ -497,7 +497,7 @@ jobs:
497497
exclude:
498498
# skip the ARM integration tests in forks, and also if not on master/upgrade-dependencies and forceARMTests is not set to true
499499
# TODO ARM runners are not yet available for private repositories; skip them for potential private forks
500-
- arch: ${{ ((github.repository != 'localstack/localstack') || (github.ref != 'refs/heads/master' && github.ref != 'upgrade-dependencies' && inputs.forceARMTests == false)) && 'arm64' || ''}}
500+
- arch: ${{ ((github.repository != 'localstack/localstack') || (github.ref != 'refs/heads/master' && !startsWith(github.ref, 'refs/tags/v') && github.ref != 'upgrade-dependencies' && inputs.forceARMTests == false)) && 'arm64' || ''}}
501501
needs:
502502
- test-integration
503503
- test-bootstrap
@@ -543,7 +543,7 @@ jobs:
543543
exclude:
544544
# skip the ARM integration tests in forks, and also if not on master/upgrade-dependencies and forceARMTests is not set to true
545545
# TODO ARM runners are not yet available for private repositories; skip them for potential private forks
546-
- runner: ${{ ((github.repository != 'localstack/localstack') || (github.ref != 'refs/heads/master' && github.ref != 'upgrade-dependencies' && inputs.forceARMTests == false)) && 'ubuntu-24.04-arm' || ''}}
546+
- runner: ${{ ((github.repository != 'localstack/localstack') || (github.ref != 'refs/heads/master' && !startsWith(github.ref, 'refs/tags/v') && github.ref != 'upgrade-dependencies' && inputs.forceARMTests == false)) && 'ubuntu-24.04-arm' || ''}}
547547
fail-fast: false
548548
runs-on: ${{ matrix.runner }}
549549
env:
@@ -618,7 +618,7 @@ jobs:
618618
exclude:
619619
# skip the ARM integration tests in forks, and also if not on master/upgrade-dependencies and forceARMTests is not set to true
620620
# TODO ARM runners are not yet available for private repositories; skip them for potential private forks
621-
- arch: ${{ ((github.repository != 'localstack/localstack') || (github.ref != 'refs/heads/master' && github.ref != 'upgrade-dependencies' && inputs.forceARMTests == false)) && 'arm64' || ''}}
621+
- arch: ${{ ((github.repository != 'localstack/localstack') || (github.ref != 'refs/heads/master' && !startsWith(github.ref, 'refs/tags/v') && github.ref != 'upgrade-dependencies' && inputs.forceARMTests == false)) && 'arm64' || ''}}
622622
needs:
623623
- test-acceptance
624624
runs-on: ubuntu-latest
@@ -647,7 +647,7 @@ jobs:
647647

648648
test-cloudwatch-v1:
649649
name: Test CloudWatch V1
650-
if: ${{ !inputs.onlyAcceptanceTests && (github.ref == 'refs/heads/master' || needs.test-preflight.outputs.cloudwatch-v1 == 'true') }}
650+
if: ${{ !inputs.onlyAcceptanceTests && (github.ref == 'refs/heads/master' || startsWith(github.ref, 'refs/tags/v') || needs.test-preflight.outputs.cloudwatch-v1 == 'true') }}
651651
runs-on: ubuntu-latest
652652
needs:
653653
- test-preflight
@@ -697,7 +697,7 @@ jobs:
697697

698698
test-ddb-v2:
699699
name: Test DynamoDB(Streams) v2
700-
if: ${{ !inputs.onlyAcceptanceTests && (github.ref == 'refs/heads/master' || needs.test-preflight.outputs.dynamodb-v2 == 'true') }}
700+
if: ${{ !inputs.onlyAcceptanceTests && (github.ref == 'refs/heads/master' || startsWith(github.ref, 'refs/tags/v') || needs.test-preflight.outputs.dynamodb-v2 == 'true') }}
701701
runs-on: ubuntu-latest
702702
needs:
703703
- test-preflight
@@ -746,7 +746,7 @@ jobs:
746746

747747
test-events-v1:
748748
name: Test EventBridge v1
749-
if: ${{ !inputs.onlyAcceptanceTests && (github.ref == 'refs/heads/master' || needs.test-preflight.outputs.events-v1 == 'true') }}
749+
if: ${{ !inputs.onlyAcceptanceTests && (github.ref == 'refs/heads/master' || startsWith(github.ref, 'refs/tags/v') || needs.test-preflight.outputs.events-v1 == 'true') }}
750750
runs-on: ubuntu-latest
751751
needs:
752752
- test-preflight
@@ -795,7 +795,7 @@ jobs:
795795

796796
test-cfn-v2-engine:
797797
name: Test CloudFormation Engine v2
798-
if: ${{ !inputs.onlyAcceptanceTests && ( github.ref == 'refs/heads/master' || needs.test-preflight.outputs.cloudformation-v2 == 'true' )}}
798+
if: ${{ !inputs.onlyAcceptanceTests && ( github.ref == 'refs/heads/master' || startsWith(github.ref, 'refs/tags/v') || needs.test-preflight.outputs.cloudformation-v2 == 'true' )}}
799799
runs-on: ubuntu-latest
800800
needs:
801801
- test-preflight
@@ -890,7 +890,7 @@ jobs:
890890

891891
capture-not-implemented:
892892
name: "Capture Not Implemented"
893-
if: ${{ !inputs.onlyAcceptanceTests && github.ref == 'refs/heads/master' }}
893+
if: ${{ !inputs.onlyAcceptanceTests && ( github.ref == 'refs/heads/master' || startsWith(github.ref, 'refs/tags/v') ) }}
894894
runs-on: ubuntu-latest
895895
needs: build
896896
env:

0 commit comments

Comments
 (0)