Merge "Also configure Ironic for UC minions"

environments/undercloud/undercloud-minion.yaml

@@ -16,6 +16,82 @@ parameter_defaults:
   # Type: boolean
   EnablePackageInstall: True
 
+  # Type of disk cleaning before and between deployments, "full" for full cleaning, "metadata" to clean only disk metadata (partition table).
+  # Type: string
+  IronicCleaningDiskErase: metadata
+
+  # Name or UUID of the *overcloud* network used for cleaning bare metal nodes. The default value of "provisioning" can be left during the initial deployment (when no networks are created yet) and should be changed to an actual UUID in a post-deployment stack update.
+  # Type: string
+  IronicCleaningNetwork: ctlplane
+
+  # Inspect interface implementation to use by default. Leave empty to use the hardware type default.
+  # Type: string
+  IronicDefaultInspectInterface: inspector
+
+  # Whether to enable use of staging drivers.
+  # Type: boolean
+  IronicEnableStagingDrivers: True
+
+  # Enabled boot interface implementations. Each hardware type must have at least one valid implementation enabled.
+  # Type: comma_delimited_list
+  IronicEnabledBootInterfaces: ['pxe', 'ilo-pxe']
+
+  # Enabled console interface implementations. Each hardware type must have at least one valid implementation enabled.
+  # Type: comma_delimited_list
+  IronicEnabledConsoleInterfaces: ['ipmitool-socat', 'ilo', 'no-console']
+
+  # Enabled deploy interface implementations. Each hardware type must have at least one valid implementation enabled.
+  # Type: comma_delimited_list
+  IronicEnabledDeployInterfaces: ['iscsi', 'direct', 'ansible']
+
+  # Enabled Ironic hardware types
+  # Type: comma_delimited_list
+  IronicEnabledHardwareTypes: ['ipmi', 'redfish', 'idrac', 'ilo']
+
+  # Enabled inspect interface implementations. Each hardware type must have at least one valid implementation enabled.
+  # Type: comma_delimited_list
+  IronicEnabledInspectInterfaces: ['inspector', 'no-inspect']
+
+  # Enabled management interface implementations. Each hardware type must have at least one valid implementation enabled.
+  # Type: comma_delimited_list
+  IronicEnabledManagementInterfaces: ['ipmitool', 'redfish', 'idrac', 'ilo']
+
+  # Enabled network interface implementations. Each hardware type must have at least one valid implementation enabled.
+  # Type: comma_delimited_list
+  IronicEnabledNetworkInterfaces: ['flat']
+
+  # Enabled power interface implementations. Each hardware type must have at least one valid implementation enabled.
+  # Type: comma_delimited_list
+  IronicEnabledPowerInterfaces: ['ipmitool', 'redfish', 'idrac', 'ilo']
+
+  # Enabled RAID interface implementations. Each hardware type must have at least one valid implementation enabled.
+  # Type: comma_delimited_list
+  IronicEnabledRaidInterfaces: ['no-raid']
+
+  # Enabled rescue interface implementations. Each hardware type must have at least one valid implementation enabled.
+  # Type: comma_delimited_list
+  IronicEnabledRescueInterfaces: ['no-rescue', 'agent']
+
+  # Enabled storage interface implementations. Each hardware type must have at least one valid implementation enabled.
+  # Type: comma_delimited_list
+  IronicEnabledStorageInterfaces: ['noop']
+
+  # Enabled vendor interface implementations. Each hardware type must have at least one valid implementation enabled.
+  # Type: comma_delimited_list
+  IronicEnabledVendorInterfaces: ['ipmitool', 'idrac', 'no-vendor']
+
+  # Whether to force power state during sync.
+  # Type: boolean
+  IronicForcePowerStateDuringSync: False
+
+  # Name or UUID of the *overcloud* network used for provisioning of bare metal nodes, if IronicDefaultNetworkInterface is set to "neutron". The default value of "provisioning" can be left during the initial deployment (when no networks are created yet) and should be changed to an actual UUID in a post-deployment stack update.
+  # Type: string
+  IronicProvisioningNetwork: ctlplane
+
+  # Name or UUID of the *overcloud* network used for resucing of bare metal nodes, if IronicDefaultRescueInterface is not set to "no-rescue". The default value of "provisioning" can be left during the initial deployment (when no networks are created yet) and should be changed to an actual UUID in a post-deployment stack update.
+  # Type: string
+  IronicRescuingNetwork: ctlplane
+
   # NTP servers list. Defaulted to a set of pool.ntp.org servers in order to have a sane default for Pacemaker deployments when not configuring this parameter by default.
   # Type: comma_delimited_list
   NtpServer: ['0.pool.ntp.org', '1.pool.ntp.org', '2.pool.ntp.org', '3.pool.ntp.org']

sample-env-generator/undercloud-minion.yaml

@@ -26,6 +26,27 @@ environments:
       overcloud.yaml:
         parameters:
           - DnsServers
+      deployment/ironic/ironic-conductor-container-puppet.yaml:
+        parameters:
+          - IronicCleaningDiskErase
+          - IronicCleaningNetwork
+          - IronicDefaultInspectInterface
+          - IronicEnabledBootInterfaces
+          - IronicEnabledConsoleInterfaces
+          - IronicEnabledDeployInterfaces
+          - IronicEnabledHardwareTypes
+          - IronicEnabledInspectInterfaces
+          - IronicEnabledManagementInterfaces
+          - IronicEnabledNetworkInterfaces
+          - IronicEnabledPowerInterfaces
+          - IronicEnabledRaidInterfaces
+          - IronicEnabledRescueInterfaces
+          - IronicEnabledStorageInterfaces
+          - IronicEnabledVendorInterfaces
+          - IronicEnableStagingDrivers
+          - IronicForcePowerStateDuringSync
+          - IronicProvisioningNetwork
+          - IronicRescuingNetwork
     sample_values:
       # Normal bits for standalone
       StackAction: CREATE
@@ -34,6 +55,28 @@ environments:
       SshFirewallAllowAll: true
       HeatEngineOptVolumes:
         - /usr/lib/heat:/usr/lib/heat:ro
+      IronicCleaningDiskErase: 'metadata'
+      IronicCorsAllowedOrigin: '*'
+      IronicDefaultInspectInterface: 'inspector'
+      IronicDefaultResourceClass: 'baremetal'
+      IronicEnabledHardwareTypes: ['ipmi', 'redfish', 'idrac', 'ilo']
+      IronicEnabledBootInterfaces: ['pxe', 'ilo-pxe']
+      IronicEnabledConsoleInterfaces: ['ipmitool-socat', 'ilo', 'no-console']
+      IronicEnabledDeployInterfaces: ['iscsi', 'direct', 'ansible']
+      IronicEnabledInspectInterfaces: ['inspector', 'no-inspect']
+      IronicEnabledManagementInterfaces: ['ipmitool', 'redfish', 'idrac', 'ilo']
+      IronicEnabledNetworkInterfaces: ['flat']
+      IronicEnabledPowerInterfaces: ['ipmitool', 'redfish', 'idrac', 'ilo']
+      IronicEnabledRaidInterfaces: ['no-raid']
+      IronicEnabledStorageInterfaces: ['noop']
+      IronicEnabledVendorInterfaces: ['ipmitool', 'idrac', 'no-vendor']
+      IronicEnableStagingDrivers: true
+      IronicCleaningNetwork: 'ctlplane'
+      IronicForcePowerStateDuringSync: false
+      IronicInspectorCollectors: default,extra-hardware,numa-topology,logs
+      IronicInspectorInterface: br-ctlplane
+      IronicProvisioningNetwork: 'ctlplane'
+      IronicRescuingNetwork: 'ctlplane'
     resource_registry:
       # this network config is assumed by the tripleo deploy command
       OS::TripleO::UndercloudMinion::Net::SoftwareConfig: ../../net-config-undercloud.yaml