From d2f6a3be26b2de2bb6aabf9e5bd3accfacc01916 Mon Sep 17 00:00:00 2001
From: Martin Schuppert <mschuppert@redhat.com>
Date: Thu, 25 Mar 2021 17:05:57 +0100
Subject: [PATCH] Remove no longer used NovaNfsEnabled parameter and condtion

With Id5503ed274bd5dc0c5365cc994de7e5cdcbc2fb6 the handling of
/var/lib/nova on NFS changed and NovaNfsEnabled and nova_nfs_enabled
condition is no longer used in most of the deployment files.

This cleans up the no longer used parameter and condition in
nova-compute-container-puppet.yaml, nova-ironic-container-puppet.yaml
nova-libvirt-container-puppet.yaml and nova-migration-target-container-puppet.yaml

Change-Id: I542956aef6da63a376790ec87d4767196ff85067
---
 deployment/nova/nova-compute-container-puppet.yaml |  7 -------
 deployment/nova/nova-ironic-container-puppet.yaml  | 12 ------------
 deployment/nova/nova-libvirt-container-puppet.yaml | 13 -------------
 .../nova-migration-target-container-puppet.yaml    | 14 --------------
 4 files changed, 46 deletions(-)

diff --git a/deployment/nova/nova-compute-container-puppet.yaml b/deployment/nova/nova-compute-container-puppet.yaml
index 6b25258894..0b1dca6a0c 100644
--- a/deployment/nova/nova-compute-container-puppet.yaml
+++ b/deployment/nova/nova-compute-container-puppet.yaml
@@ -989,13 +989,6 @@ conditions:
 
   is_not_additional_cell: {equals: [{get_param: NovaAdditionalCell}, false]}
 
-  nova_nfs_enabled:
-    or:
-      - and:
-        - equals: [{get_param: NovaNfsEnabled}, true]
-        - equals: [{get_param: [RoleParameters, NovaNfsEnabled]}, '']
-      - equals: [{get_param: [RoleParameters, NovaNfsEnabled]}, true]
-
   reserved_huge_pages_set:
     not:
       and:
diff --git a/deployment/nova/nova-ironic-container-puppet.yaml b/deployment/nova/nova-ironic-container-puppet.yaml
index 1043e6559c..96bfbfd804 100644
--- a/deployment/nova/nova-ironic-container-puppet.yaml
+++ b/deployment/nova/nova-ironic-container-puppet.yaml
@@ -45,12 +45,6 @@ parameters:
     default: false
     description: Whether to enable the multipath daemon
     type: boolean
-  NovaNfsEnabled:
-    default: false
-    description: Whether to enable or not the NFS backend for Nova
-    type: boolean
-    tags:
-      - role_specific
   IronicApiMaxRetries:
     description: The number of times to retry when a request conflicts. If set to 0, only try once, no retries.
     type: number
@@ -66,12 +60,6 @@ parameters:
 
 conditions:
 
-  nova_nfs_enabled:
-    or:
-      - and:
-        - equals: [{get_param: NovaNfsEnabled}, true]
-        - equals: [{get_param: [RoleParameters, NovaNfsEnabled]}, '']
-      - equals: [{get_param: [RoleParameters, NovaNfsEnabled]}, true]
   ironic_api_max_retry_cond:
     equals: [{get_param: IronicApiMaxRetries}, -1]
 
diff --git a/deployment/nova/nova-libvirt-container-puppet.yaml b/deployment/nova/nova-libvirt-container-puppet.yaml
index cb53604fac..938c3a653a 100644
--- a/deployment/nova/nova-libvirt-container-puppet.yaml
+++ b/deployment/nova/nova-libvirt-container-puppet.yaml
@@ -229,12 +229,6 @@ parameters:
     description: The password for the libvirt service when TLS is enabled
     type: string
     hidden: true
-  NovaNfsEnabled:
-    default: false
-    description: Whether to enable or not the NFS backend for Nova
-    type: boolean
-    tags:
-      - role_specific
   LibvirtLogFilters:
     description: Defines a filter in libvirt daemon to select a different
                  logging level for a given category log outputs, as specified
@@ -339,13 +333,6 @@ conditions:
       - {get_param: QemuCACert}
       - ''
 
-  nova_nfs_enabled:
-    or:
-      - and:
-        - equals: [{get_param: NovaNfsEnabled}, true]
-        - equals: [{get_param: [RoleParameters, NovaNfsEnabled]}, '']
-      - equals: [{get_param: [RoleParameters, NovaNfsEnabled]}, true]
-
   key_size_libvirt_override_unset: {equals: [{get_param: LibvirtCertificateKeySize}, '']}
   key_size_libvirtvnc_override_unset: {equals: [{get_param: LibvirtVNCServerCertificateKeySize}, '']}
   key_size_qemu_client_override_unset: {equals: [{get_param: QemuClientCertificateKeySize}, '']}
diff --git a/deployment/nova/nova-migration-target-container-puppet.yaml b/deployment/nova/nova-migration-target-container-puppet.yaml
index 3b7dbe6d2c..88331b2c0c 100644
--- a/deployment/nova/nova-migration-target-container-puppet.yaml
+++ b/deployment/nova/nova-migration-target-container-puppet.yaml
@@ -51,12 +51,6 @@ parameters:
     default: 2022
     description: Target port for migration over ssh
     type: number
-  NovaNfsEnabled:
-    default: false
-    description: Whether to enable or not the NFS backend for Nova
-    type: boolean
-    tags:
-      - role_specific
 
 resources:
 
@@ -71,14 +65,6 @@ resources:
       RoleName: {get_param: RoleName}
       RoleParameters: {get_param: RoleParameters}
 
-conditions:
-  nova_nfs_enabled:
-    or:
-      - and:
-        - equals: [{get_param: NovaNfsEnabled}, true]
-        - equals: [{get_param: [RoleParameters, NovaNfsEnabled]}, '']
-      - equals: [{get_param: [RoleParameters, NovaNfsEnabled]}, true]
-
 outputs:
   role_data:
     description: Role data for the Nova Migration Target service.