Make image-build infra management optional

Author: m-bull

build-image.yml

@@ -0,0 +1,31 @@
+---
+- name: Build image
+  hosts: openstack
+  tasks:
+    - name: Create image build infra
+      include_role:
+        name: image_build_infra
+        tasks_from: create.yml
+      when:
+        - image_build_manage_infra is defined
+        - image_build_manage_infra
+
+    - name: Build fat image
+      include_role:
+        name: image_build
+
+    - name: Destroy image build infra
+      include_role:
+        name: image_build_infra
+        tasks_from: destroy.yml
+      when:
+        - image_build_manage_infra is defined
+        - image_build_manage_infra
+
+    - name: Set cluster_image fact
+      set_fact:
+        cluster_image: "{{ image_build_data.artifact_id }}"
+
+    - name: Print cluster_image UUID
+      debug:
+        msg: "{{ cluster_image }}"

image_build/hosts renamed to image-build/hosts

@@ -1,53 +1,8 @@
 ---
-- debug:
-    msg: |
-      terraform_backend_type: {{ terraform_backend_type }}
-      terraform_state: {{ terraform_state }}
 
 - name: Run prechecks
   include_tasks: prechecks.yml
 
-- name: Install Terraform binary
-  include_role:
-    name: stackhpc.terraform.install
-
-- name: Make Terraform project directory
-  file:
-    path: "{{ image_build_terraform_project_path }}"
-    state: directory
-
-- name: Write backend configuration
-  copy:
-    content: |
-      terraform {
-        backend "{{ terraform_backend_type }}" { }
-      }
-    dest: "{{ image_build_terraform_project_path }}/backend.tf"
-
-- name: Template Terraform files into project directory
-  template:
-    src: "{{ item }}.j2"
-    dest: "{{ image_build_terraform_project_path }}/{{ item }}"
-  loop:
-    - outputs.tf
-    - providers.tf
-    - resources.tf
-
-- name: Provision infrastructure
-  include_role:
-    name: stackhpc.terraform.infra
-  vars:
-    terraform_project_path: "{{ image_build_terraform_project_path }}"
-
-- name: Set image build infrastructure facts
-  set_fact:
-    image_build_network_id: "{{ terraform_provision.outputs.network_id.value }}"
-    image_build_floating_ip_network: "{{ terraform_provision.outputs.floating_ip_network_id.value }}"
-    image_build_ssh_keypair_name: "{{ terraform_provision.outputs.ssh_keypair_name.value }}"
-    image_build_ssh_private_key_file: "{{ cluster_ssh_private_key_file }}"
-    image_build_source_image_id: "{{ terraform_provision.outputs.source_image_name.value.ids | first }}"
-    image_build_security_group_id: "{{ terraform_provision.outputs.security_group_id.value }}"
-
 - name: Make Packer vars file
   template:
     src: builder.pkrvars.hcl.j2
@@ -118,10 +73,3 @@
   set_fact:
     image_build_data: "{{ packer_manifest.builds | selectattr('packer_run_uuid', 'eq', packer_manifest.last_run_uuid) | first }}"
 
-- name: Destroy infrastructure
-  include_role:
-    name: stackhpc.terraform.infra
-  vars:
-    terraform_project_path: "{{ image_build_terraform_project_path }}"
-    cluster_state: absent
-

image-build/image-build.yml

@@ -1,5 +1,18 @@
 ---
 
+- name: Check required vars are defined
+  assert:
+    that:
+      - "{{ item }} is defined"
+    fail_msg: "{{ item }} is not defined"
+  loop:
+    - image_build_network_id
+    - image_build_floating_ip_network
+    - image_build_ssh_keypair_name
+    - image_build_ssh_private_key_file
+    - image_build_source_image_id
+    - image_build_security_group_id
+
 - name: Ensure builder access mode
   fail:
     msg: >- 

roles/image_build/tasks/main.yml

@@ -0,0 +1,12 @@
+---
+image_build_terraform_project_path: "{{ playbook_dir }}/terraform-image-build"
+image_build_cluster_id: "image-build"
+
+# Regex to capture existing cloud image names to use as the
+# OpenHPC Slurm base-image
+image_build_existing_image_regex: "^Rocky-8-GenericCloud-Base-8.7-.*"
+# Attributes to sort the list of existing base images returned by
+# image_build_existing_image_regex. See
+# https://registry.terraform.io/providers/terraform-provider-openstack/openstack/latest/docs/data-sources/images_image_ids_v2#sort
+image_build_existing_image_sort_attributes: "name,updated_at"
+

roles/image_build/tasks/prechecks.yml

@@ -0,0 +1,47 @@
+---
+- debug:
+    msg: |
+      terraform_backend_type: {{ terraform_backend_type }}
+      terraform_state: {{ terraform_state }}
+
+- name: Install Terraform binary
+  include_role:
+    name: stackhpc.terraform.install
+
+- name: Make Terraform project directory
+  file:
+    path: "{{ image_build_terraform_project_path }}"
+    state: directory
+
+- name: Write backend configuration
+  copy:
+    content: |
+      terraform {
+        backend "{{ terraform_backend_type }}" { }
+      }
+    dest: "{{ image_build_terraform_project_path }}/backend.tf"
+
+- name: Template Terraform files into project directory
+  template:
+    src: "{{ item }}.j2"
+    dest: "{{ image_build_terraform_project_path }}/{{ item }}"
+  loop:
+    - outputs.tf
+    - providers.tf
+    - resources.tf
+
+- name: Provision infrastructure
+  include_role:
+    name: stackhpc.terraform.infra
+  vars:
+    terraform_project_path: "{{ image_build_terraform_project_path }}"
+
+- name: Set image build infrastructure facts
+  set_fact:
+    image_build_network_id: "{{ terraform_provision.outputs.network_id.value }}"
+    image_build_floating_ip_network: "{{ terraform_provision.outputs.floating_ip_network_id.value }}"
+    image_build_ssh_keypair_name: "{{ terraform_provision.outputs.ssh_keypair_name.value }}"
+    image_build_ssh_private_key_file: "{{ cluster_ssh_private_key_file }}"
+    image_build_source_image_id: "{{ terraform_provision.outputs.source_image_name.value.ids | first }}"
+    image_build_security_group_id: "{{ terraform_provision.outputs.security_group_id.value }}"
+

roles/image_build_infra/defaults/main.yml

@@ -0,0 +1,9 @@
+---
+
+- name: Destroy infrastructure
+  include_role:
+    name: stackhpc.terraform.infra
+  vars:
+    terraform_project_path: "{{ image_build_terraform_project_path }}"
+    cluster_state: absent
+