Skip to content

Commit e23cba8

Browse files
authored
Merge pull request #221 from ACCESS-NRI/220-consistent-environment-names
Update Acceptable Environments to the form `SUPERCOMPUTER TYPE`
2 parents 2b635fb + 23bf067 commit e23cba8

File tree

3 files changed

+7
-12
lines changed

3 files changed

+7
-12
lines changed

.github/workflows/deploy-1-setup.yml

+1-2
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ on:
88
required: true
99
description: |
1010
The target machine for the model being deployed.
11-
Equivalent to the name of the GitHub Environment, minus a potential Prerelease Type.
11+
Equivalent to the name of the GitHub Environment, minus a Release/Prerelease Type.
1212
deployment-type:
1313
type: string
1414
required: true
@@ -282,7 +282,6 @@ jobs:
282282
- check-spack-yaml # Verify spack manifest information is correct
283283
uses: access-nri/build-cd/.github/workflows/deploy-2-start.yml@v4
284284
with:
285-
type: ${{ inputs.deployment-type }}
286285
model: ${{ inputs.spack-manifest-root-sbd }}
287286
ref: ${{ inputs.deployment-ref }}
288287
version: ${{ inputs.deployment-version }}

.github/workflows/deploy-2-start.yml

+4-8
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,6 @@ concurrency: ${{ inputs.deployment-target }} ${{ inputs.deployment-type }}
33
on:
44
workflow_call:
55
inputs:
6-
type:
7-
type: string
8-
required: true
9-
description: The type of deployment - either 'Release' or 'Prerelease'
106
model:
117
type: string
128
required: true
@@ -27,15 +23,15 @@ on:
2723
type: string
2824
required: true
2925
description: |
30-
The GitHub Environment, minus a potential Type.
26+
The GitHub Environment, minus a Type.
3127
Combined with inputs.deployment-type to create the GitHub Environment name.
3228
deployment-type:
3329
type: string
3430
required: true
3531
description: |
3632
The type of deployment.
3733
Can be one of: Release, Prerelease.
38-
Combined with inputs.deployment-type to create the GitHub Environment name.
34+
Combined with inputs.deployment-target to create the GitHub Environment name.
3935
root-sbd:
4036
type: string
4137
required: true
@@ -63,7 +59,7 @@ jobs:
6359
deploy-to-environment:
6460
name: Deploy to ${{ inputs.deployment-target }} ${{ inputs.deployment-type }}
6561
runs-on: ubuntu-latest
66-
environment: ${{ inputs.deployment-target }} ${{ inputs.deployment-type != 'Release' && inputs.deployment-type || '' }}
62+
environment: ${{ inputs.deployment-target }} ${{ inputs.deployment-type }}
6763
outputs:
6864
packages-version: ${{ steps.versions.outputs.packages }}
6965
config-version: ${{ steps.versions.outputs.config }}
@@ -100,7 +96,7 @@ jobs:
10096
${{ secrets.HOST_DATA }}
10197
10298
- name: Prerelease spack.yaml Modifications
103-
if: inputs.type == 'Prerelease'
99+
if: inputs.deployment-type == 'Prerelease'
104100
# Modifies the name of the prerelease modulefile to the
105101
# `pr<number>-<commit>` style. For example, `access-om3/pr12-2`.
106102
# Also removes the `@git.VERSION` specifier for Prereleases so

.github/workflows/undeploy-1-start.yml

+2-2
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ jobs:
3838
name: Undeploy ${{ inputs.version-pattern }} from ${{ inputs.target }} ${{ inputs.type }}
3939
needs: validate-inputs
4040
runs-on: ubuntu-latest
41-
environment: ${{ inputs.target }} ${{ inputs.type != 'Release' && inputs.type }}
41+
environment: ${{ inputs.target }} ${{ inputs.type }}
4242
steps:
4343
- uses: actions/checkout@v4
4444

@@ -52,7 +52,7 @@ jobs:
5252
with:
5353
spack-installs-root-path: ${{ vars.SPACK_INSTALLS_ROOT_LOCATION }}
5454
spack-version: ${{ steps.versions.outputs.spack }}
55-
deployment-environment: ${{ inputs.target }} ${{ inputs.type != 'Release' && inputs.type }}
55+
deployment-environment: ${{ inputs.target }} ${{ inputs.type }}
5656

5757
- name: Setup SSH
5858
id: ssh

0 commit comments

Comments
 (0)