Skip to content

Commit fb25947

Browse files
authored
Merge pull request #1310 from stackhpc/container-image-build
Validate inputs to github workflows
2 parents 98a9d28 + abf4976 commit fb25947

File tree

4 files changed

+34
-2
lines changed

4 files changed

+34
-2
lines changed

.github/workflows/overcloud-host-image-build.yml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,13 @@ jobs:
3232
runs-on: arc-skc-host-image-builder-runner
3333
permissions: {}
3434
steps:
35+
- name: Validate inputs
36+
run: |
37+
if [[ ${{ inputs.rocky9 }} == 'false' && ${{ inputs.ubuntu-jammy }} == 'false' ]]; then
38+
echo "At least one distribution must be selected"
39+
exit 1
40+
fi
41+
3542
- name: Install Package
3643
uses: ConorMacBride/install-package@main
3744
with:

.github/workflows/overcloud-host-image-promote.yml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,13 @@ jobs:
2323
if: github.repository == 'stackhpc/stackhpc-kayobe-config'
2424
runs-on: ubuntu-22.04
2525
steps:
26+
- name: Validate inputs
27+
run: |
28+
if [[ ${{ inputs.rocky9 }} == 'false' && ${{ inputs.ubuntu-jammy }} == 'false' ]]; then
29+
echo "At least one distribution must be selected"
30+
exit 1
31+
fi
32+
2633
- uses: actions/checkout@v4
2734
with:
2835
path: src/kayobe-config

.github/workflows/overcloud-host-image-upload.yml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,13 @@ jobs:
3434
runs-on: arc-skc-host-image-builder-runner
3535
permissions: {}
3636
steps:
37+
- name: Validate inputs
38+
run: |
39+
if [[ ${{ inputs.rocky9 }} == 'false' && ${{ inputs.ubuntu-jammy }} == 'false' ]]; then
40+
echo "At least one distribution must be selected"
41+
exit 1
42+
fi
43+
3744
- name: Install package dependencies
3845
run: |
3946
sudo apt update

.github/workflows/stackhpc-container-image-build.yml

Lines changed: 13 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,12 +9,12 @@ on:
99
required: false
1010
default: ""
1111
overcloud:
12-
description: Build overcloud images?
12+
description: Build container images for overcloud services?
1313
type: boolean
1414
required: false
1515
default: true
1616
seed:
17-
description: Build seed images?
17+
description: Build container images for seed services?
1818
type: boolean
1919
required: false
2020
default: false
@@ -52,6 +52,17 @@ jobs:
5252
matrix: ${{ steps.set-matrix.outputs.matrix }}
5353
openstack_release: ${{ steps.openstack_release.outputs.openstack_release }}
5454
steps:
55+
- name: Validate inputs
56+
run: |
57+
if [[ ${{ inputs.rocky-linux-9 }} == 'false' && ${{ inputs.ubuntu-jammy }} == 'false' ]]; then
58+
echo "At least one distribution must be selected"
59+
exit 1
60+
fi
61+
if [[ ${{ inputs.overcloud }} == 'false' && ${{ inputs.seed }} == 'false' ]]; then
62+
echo "At least one of overcloud or seed must be selected"
63+
exit 1
64+
fi
65+
5566
- name: Checkout
5667
uses: actions/checkout@v4
5768

0 commit comments

Comments
 (0)