Skip to content

Commit aa67f2e

Browse files
CCM-11942 fixing environment refs for acceptance tests
1 parent 90c2694 commit aa67f2e

File tree

2 files changed

+19
-4
lines changed

2 files changed

+19
-4
lines changed

.github/workflows/cicd-1-pull-request.yaml

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50,12 +50,18 @@ jobs:
5050
run: |
5151
branch_name=${GITHUB_HEAD_REF:-$(echo $GITHUB_REF | sed 's#refs/heads/##')}
5252
echo "Current branch is '$branch_name'"
53-
if gh pr list --head $branch_name | grep -q .; then
54-
echo "Pull request exists"
53+
54+
pr_json=$(gh pr list --head "$branch_name" --state open --json number --limit 1)
55+
pr_number=$(echo "$pr_json" | jq -r '.[0].number // empty')
56+
57+
if [[ -n "$pr_number" ]]; then
58+
echo "Pull request exists: #$pr_number"
5559
echo "does_pull_request_exist=true" >> $GITHUB_OUTPUT
60+
echo "pr_number=$pr_number" >> $GITHUB_OUTPUT
5661
else
5762
echo "Pull request doesn't exist"
5863
echo "does_pull_request_exist=false" >> $GITHUB_OUTPUT
64+
echo "pr_number=" >> $GITHUB_OUTPUT
5965
fi
6066
- name: "List variables"
6167
run: |
@@ -68,6 +74,7 @@ jobs:
6874
export TERRAFORM_VERSION="${{ steps.variables.outputs.terraform_version }}"
6975
export VERSION="${{ steps.variables.outputs.version }}"
7076
export DOES_PULL_REQUEST_EXIST="${{ steps.pr_exists.outputs.does_pull_request_exist }}"
77+
export IS_VERSION_PRERELEASE="${{ steps.variables.outputs.is_version_prerelease }}"
7178
make list-variables
7279
commit-stage: # Recommended maximum execution time is 2 minutes
7380
name: "Commit stage"
@@ -99,4 +106,7 @@ jobs:
99106
name: "Acceptance stage"
100107
needs: [metadata, test-stage]
101108
uses: ./.github/workflows/stage-4-acceptance.yaml
109+
if: needs.metadata.outputs.does_pull_request_exist == 'true' || (github.event_name == 'pull_request' && (github.event.action == 'opened' || github.event.action == 'reopened')) || (github.event_name == 'push' && github.ref == 'refs/heads/main')
102110
secrets: inherit
111+
with:
112+
pr_number: ${{ needs.metadata.outputs.pr_number }}

.github/workflows/stage-4-acceptance.yaml

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,11 @@ name: Acceptance stage
22

33
on:
44
workflow_call:
5+
inputs:
6+
pr_number:
7+
required: true
8+
type: string
9+
510

611
permissions:
712
id-token: write
@@ -25,7 +30,7 @@ jobs:
2530
--infraRepoName "nhs-notify-web-template-management" \
2631
--releaseVersion "${GITHUB_HEAD_REF:-${GITHUB_REF#refs/heads/}}" \
2732
--targetWorkflow "dispatch-deploy-dynamic-env.yaml" \
28-
--targetEnvironment "pr${{ github.event.number }}" \
33+
--targetEnvironment "pr${{ inputs.pr_number }}" \
2934
--targetAccountGroup "nhs-notify-template-management-dev" \
3035
--targetComponent "sandbox" \
3136
--terraformAction "apply"
@@ -40,6 +45,6 @@ jobs:
4045
--infraRepoName "nhs-notify-web-template-management" \
4146
--releaseVersion "${GITHUB_HEAD_REF:-${GITHUB_REF#refs/heads/}}" \
4247
--targetWorkflow "dispatch-contextual-tests-dynamic-env.yaml" \
43-
--targetEnvironment "${{ needs.generate-sandbox-name.outputs.SANDBOX_NAME }}" \
48+
--targetEnvironment "pr${{ inputs.pr_number }}" \
4449
--targetAccountGroup "nhs-notify-template-management-dev" \
4550
--targetComponent "sandbox"

0 commit comments

Comments
 (0)