Skip to content

Commit cff0dcc

Browse files
authored
ci: add label condition to integration-test-deployment.yml (#35708)
### Reason for this change The new integ test deployment workflow asks for a deployment in the UI. This has been confusing for contributors as it is similarly named to the general integ test available today. ### Description of changes - Skip this job if the label `request-deployment-integ-run` - Update name of the job to clarify its purpose ### Describe any new or updated permissions being added No new permissions added. ### Checklist - [x] My code adheres to the [CONTRIBUTING GUIDE](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md) and [DESIGN GUIDELINES](https://github.com/aws/aws-cdk/blob/main/docs/DESIGN_GUIDELINES.md) ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
1 parent 57f9068 commit cff0dcc

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

.github/workflows/integration-test-deployment.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,8 @@ jobs:
2121
integration_test_deployment:
2222
runs-on: codebuild-aws-cdk-github-actions-deployment-integ-runner-${{ github.run_id }}-${{ github.run_attempt }}
2323
environment: deployment-integ-test # Do not change or remove this without discussing with Appsec
24+
if: contains(github.event.pull_request.labels.*.name, 'pr/needs-integration-tests-deployment ')
25+
name: 'Deploy integration test snapshots (requires `pr/needs-integration-tests-deployment` label)'
2426

2527
env:
2628
PR_BUILD: true

0 commit comments

Comments
 (0)