From 8e930b286aec7ca49ddfcb2ca0b9b92f2ac031e1 Mon Sep 17 00:00:00 2001 From: Judd Maltin Date: Tue, 24 Mar 2026 08:53:01 -0400 Subject: [PATCH] =?UTF-8?q?fix:=20correct=20typo=20boot=5Fsources=5Fshapsh?= =?UTF-8?q?ot=20=E2=86=92=20boot=5Fsources=5Fsnapshot?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The variable ocp4_workload_openshift_virtualization_boot_sources_shapshot was misspelled ("shapshot" instead of "snapshot") in defaults, tasks, and template. No existing catalog items use this v2 variable name, so this is a safe rename with no breaking changes. Co-Authored-By: Claude Opus 4.6 --- roles/ocp4_workload_openshift_virtualization/defaults/main.yml | 2 +- roles/ocp4_workload_openshift_virtualization/tasks/workload.yml | 2 +- .../templates/hyperconverged.yaml.j2 | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/roles/ocp4_workload_openshift_virtualization/defaults/main.yml b/roles/ocp4_workload_openshift_virtualization/defaults/main.yml index 90fe1ab..5f62d57 100644 --- a/roles/ocp4_workload_openshift_virtualization/defaults/main.yml +++ b/roles/ocp4_workload_openshift_virtualization/defaults/main.yml @@ -30,7 +30,7 @@ ocp4_workload_openshift_virtualization_workload_tolerations: [] # For deployments on top of CNV with Ceph set to false # When set to false CNV is configured to use PVCs for boot sources # which elminiates VolumeSnapshots without associated PVCs -ocp4_workload_openshift_virtualization_boot_sources_shapshot: true +ocp4_workload_openshift_virtualization_boot_sources_snapshot: true # For AWS it is possible to use KubeVirt without a metal node # For that the subscription for the operator needs to have special config diff --git a/roles/ocp4_workload_openshift_virtualization/tasks/workload.yml b/roles/ocp4_workload_openshift_virtualization/tasks/workload.yml index 3bd6ea2..03084ff 100644 --- a/roles/ocp4_workload_openshift_virtualization/tasks/workload.yml +++ b/roles/ocp4_workload_openshift_virtualization/tasks/workload.yml @@ -54,7 +54,7 @@ # HyperConverged must have been created with .spec.featureGates.enableCommonBootImageImport: false # See template for that logic - name: Set boot sources to use PVCs - when: not ocp4_workload_openshift_virtualization_boot_sources_shapshot | bool + when: not ocp4_workload_openshift_virtualization_boot_sources_snapshot | bool block: - name: Get all available storage classes kubernetes.core.k8s_info: diff --git a/roles/ocp4_workload_openshift_virtualization/templates/hyperconverged.yaml.j2 b/roles/ocp4_workload_openshift_virtualization/templates/hyperconverged.yaml.j2 index 6b15b3c..34f1545 100644 --- a/roles/ocp4_workload_openshift_virtualization/templates/hyperconverged.yaml.j2 +++ b/roles/ocp4_workload_openshift_virtualization/templates/hyperconverged.yaml.j2 @@ -11,7 +11,7 @@ spec: tolerations: {{ ocp4_workload_openshift_virtualization_workload_tolerations | to_yaml }} {% endif %} -{% if ocp4_workload_openshift_virtualization_boot_sources_shapshot | default(true) | bool %} +{% if ocp4_workload_openshift_virtualization_boot_sources_snapshot | default(true) | bool %} enableCommonBootImageImport: true {% else %} enableCommonBootImageImport: false