Skip to content

Commit e942323

Browse files
stevebuxstevebux
committed
Experimental commit
1 parent 6b9c7a1 commit e942323

3 files changed

Lines changed: 86 additions & 54 deletions

File tree

‎.github/actions/build-oas-spec/action.yml‎

Lines changed: 71 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,71 @@
1+
name: "Build Proxies"
2+
description: "Build Proxies"
3+
4+
inputs:
5+
apimEnv:
6+
description: "APIM environment"
7+
required: true
8+
buildSandbox:
9+
description: "Whether to build the sandbox OAS spec"
10+
required: false
11+
default: false
12+
nodejs_version:
13+
description: "Node.js version, set by the CI/CD pipeline workflow"
14+
required: true
15+
NODE_AUTH_TOKEN:
16+
description: "Token for access to github package registry"
17+
required: true
18+
19+
runs:
20+
using: composite
21+
22+
steps:
23+
- name: Checkout
24+
uses: actions/checkout@v4
25+
- uses: actions/setup-node@v4
26+
with:
27+
node-version: ${{ inputs.nodejs_version }}
28+
registry-url: 'https://npm.pkg.github.com'
29+
30+
- name: "Cache node_modules"
31+
uses: actions/cache@v4
32+
with:
33+
path: |
34+
**/node_modules
35+
key: ${{ runner.os }}-node-${{ inputs.nodejs_version }}-${{ hashFiles('**/package-lock.json') }}
36+
restore-keys: |
37+
${{ runner.os }}-node-${{ inputs.nodejs_version }}-
38+
39+
- name: Npm install
40+
working-directory: .
41+
env:
42+
NODE_AUTH_TOKEN: ${{ inputs.NODE_AUTH_TOKEN }}
43+
run: npm ci
44+
shell: bash
45+
46+
- name: Build ${{ inputs.apimEnv }} oas
47+
working-directory: .
48+
env:
49+
APIM_ENV: ${{ inputs.apimEnv }}
50+
shell: bash
51+
run: |
52+
if [ ${{ env.APIM_ENV }} == "internal-dev-sandbox" ] && [ ${{ inputs.buildSandbox }} == true ]
53+
then
54+
echo "Building sandbox OAS spec"
55+
make build-json-oas-spec APIM_ENV=sandbox
56+
else
57+
echo "Building env specific OAS spec"
58+
make build-json-oas-spec APIM_ENV=${{ env.APIM_ENV }}
59+
fi
60+
61+
if [[ $APIM_ENV == *-pr ]]; then
62+
echo "Removing pr suffix from APIM_ENV after building OAS and calling proxygen"
63+
APIM_ENV=$(echo "$APIM_ENV" | sed 's/-pr$//')
64+
fi
65+
echo "APIM_ENV=$APIM_ENV" >> $GITHUB_ENV
66+
67+
- name: Upload OAS Spec
68+
uses: actions/upload-artifact@v4
69+
with:
70+
name: ${{ env.APIM_ENV }}-build-output
71+
path: ./build

‎.github/actions/build-proxies/action.yml‎

Lines changed: 0 additions & 50 deletions
Original file line numberDiff line numberDiff line change
@@ -25,40 +25,11 @@ inputs:
2525
description: "Name of the Component to deploy"
2626
required: true
2727
default: 'api'
28-
nodejs_version:
29-
description: "Node.js version, set by the CI/CD pipeline workflow"
30-
required: true
31-
NODE_AUTH_TOKEN:
32-
description: "Token for access to github package registry"
33-
required: true
3428

3529
runs:
3630
using: composite
3731

3832
steps:
39-
- name: Checkout
40-
uses: actions/checkout@v4
41-
- uses: actions/setup-node@v4
42-
with:
43-
node-version: ${{ inputs.nodejs_version }}
44-
registry-url: 'https://npm.pkg.github.com'
45-
46-
- name: "Cache node_modules"
47-
uses: actions/cache@v4
48-
with:
49-
path: |
50-
**/node_modules
51-
key: ${{ runner.os }}-node-${{ inputs.nodejs_version }}-${{ hashFiles('**/package-lock.json') }}
52-
restore-keys: |
53-
${{ runner.os }}-node-${{ inputs.nodejs_version }}-
54-
55-
- name: Npm install
56-
working-directory: .
57-
env:
58-
NODE_AUTH_TOKEN: ${{ inputs.NODE_AUTH_TOKEN }}
59-
run: npm ci
60-
shell: bash
61-
6233
- name: Setup Proxy Name and target
6334
shell: bash
6435
run: |
@@ -87,27 +58,6 @@ runs:
8758
echo "MTLS_NAME=notify-supplier-mtls-pr$PR_NUMBER" >> $GITHUB_ENV
8859
fi
8960
90-
- name: Build ${{ inputs.apimEnv }} oas
91-
working-directory: .
92-
env:
93-
APIM_ENV: ${{ inputs.apimEnv }}
94-
shell: bash
95-
run: |
96-
if [ ${{ env.APIM_ENV }} == "internal-dev-sandbox" ] && [ ${{ inputs.buildSandbox }} == true ]
97-
then
98-
echo "Building sandbox OAS spec"
99-
make build-json-oas-spec APIM_ENV=sandbox
100-
else
101-
echo "Building env specific OAS spec"
102-
make build-json-oas-spec APIM_ENV=${{ env.APIM_ENV }}
103-
fi
104-
105-
if [[ $APIM_ENV == *-pr ]]; then
106-
echo "Removing pr suffix from APIM_ENV after building OAS and calling proxygen"
107-
APIM_ENV=$(echo "$APIM_ENV" | sed 's/-pr$//')
108-
fi
109-
echo "APIM_ENV=$APIM_ENV" >> $GITHUB_ENV
110-
11161
- name: Upload OAS Spec
11262
uses: actions/upload-artifact@v4
11363
with:

‎.github/workflows/stage-3-build.yaml‎

Lines changed: 15 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -114,6 +114,21 @@ jobs:
114114
--terraformAction "apply" \
115115
--overrideProjectName "nhs" \
116116
--overrideRoleName "nhs-main-acct-supplier-api-github-deploy"
117+
artefact-oas-spec:
118+
name: "Build OAS spec"
119+
runs-on: ubuntu-latest
120+
needs: [artefact-jekyll-docs]
121+
timeout-minutes: 10
122+
steps:
123+
- name: "Checkout code"
124+
uses: actions/checkout@v5
125+
- name: "Build proxies"
126+
uses: ./.github/actions/build-oas-spec
127+
with:
128+
apimEnv: "internal-dev-sandbox"
129+
buildSandbox: true
130+
nodejs_version: ${{ inputs.nodejs_version }}
131+
NODE_AUTH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
117132
artefact-proxies:
118133
name: "Build proxies"
119134
runs-on: ubuntu-latest
@@ -125,8 +140,6 @@ jobs:
125140
APP_CLIENT_ID: ${{ secrets.APP_CLIENT_ID }}
126141
APP_PEM_FILE: ${{ secrets.APP_PEM_FILE }}
127142
steps:
128-
- name: "Checkout code"
129-
uses: actions/checkout@v5
130143
- name: "Build proxies"
131144
uses: ./.github/actions/build-proxies
132145
with:
@@ -136,5 +149,3 @@ jobs:
136149
runId: "${{ github.run_id }}"
137150
buildSandbox: true
138151
releaseVersion: ${{ github.head_ref || github.ref_name }}
139-
nodejs_version: ${{ inputs.nodejs_version }}
140-
NODE_AUTH_TOKEN: ${{ secrets.GITHUB_TOKEN }}

0 commit comments

Comments
 (0)