Skip to content

Commit 354b048

Browse files
committed
additional PR comments
1 parent cf417e5 commit 354b048

File tree

3 files changed

+8
-13
lines changed

3 files changed

+8
-13
lines changed

.github/workflows/fatimage.yml

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -88,16 +88,15 @@ jobs:
8888
cd packer/
8989
packer init .
9090
91-
SOURCE_IMAGE=${{ fromJSON(env.SOURCE_IMAGES_MAP)[matrix.os_version][matrix.build] }}
92-
9391
PACKER_LOG=1 packer build \
9492
-on-error=${{ vars.PACKER_ON_ERROR }} \
9593
-only=${{ matrix.build }} \
9694
-var-file=$PKR_VAR_environment_root/${{ env.CI_CLOUD }}.pkrvars.hcl \
97-
-var "source_image_name=${SOURCE_IMAGE}" \
95+
-var "source_image_name=${{ env.SOURCE_IMAGE }}" \
9896
openstack.pkr.hcl
9997
env:
10098
PKR_VAR_os_version: ${{ matrix.os_version }}
99+
SOURCE_IMAGE: ${{ fromJSON(env.SOURCE_IMAGES_MAP)[matrix.os_version][matrix.build] }}
101100

102101
- name: Get created image names from manifest
103102
id: manifest

.github/workflows/nightlybuild.yml

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -85,17 +85,16 @@ jobs:
8585
cd packer/
8686
packer init .
8787
88-
SOURCE_IMAGE=${{ fromJSON(env.SOURCE_IMAGES_MAP)[matrix.os_version] }}
89-
9088
PACKER_LOG=1 packer build \
9189
-on-error=${{ vars.PACKER_ON_ERROR }} \
9290
-only=${{ matrix.build }} \
9391
-var-file=$PKR_VAR_environment_root/${{ env.CI_CLOUD }}.pkrvars.hcl \
94-
-var "source_image_name=${SOURCE_IMAGE}"
92+
-var "source_image_name=${{ env.SOURCE_IMAGE }}"
9593
openstack.pkr.hcl
9694
9795
env:
9896
PKR_VAR_os_version: ${{ matrix.os_version }}
97+
SOURCE_IMAGE: ${{ fromJSON(env.SOURCE_IMAGES_MAP)[matrix.os_version] }}
9998

10099
- name: Get created image names from manifest
101100
id: manifest

packer/openstack.pkr.hcl

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -52,12 +52,9 @@ variable "source_image_name" {
5252
}
5353

5454
variable "source_image" {
55-
type = map(string)
56-
default = {
57-
RL8: null
58-
RL9: null
59-
}
60-
description = "UUID of source image, keyed from var.os_version"
55+
type = string
56+
default = null
57+
description = "UUID of source image"
6158
}
6259

6360
variable "flavor" {
@@ -169,7 +166,7 @@ source "openstack" "openhpc" {
169166
security_groups = var.security_groups
170167

171168
# Input image:
172-
source_image = "${var.source_image[var.os_version]}"
169+
source_image = "${var.source_image}"
173170
source_image_name = "${var.source_image_name}" # NB: must already exist in OpenStack
174171

175172
# SSH:

0 commit comments

Comments
 (0)