Merge "Add IronicEnableNovaPowerNotifications parameter"

deployment/ironic/ironic-conductor-container-puppet.yaml

@@ -303,6 +303,11 @@ parameters:
     description: Whether to use SNP (Simple Network Protocol) iPXE EFI, or not.
                  When set to true `ipxe-snponly` EFI is used.
     default: true
+  IronicEnableNovaPowerNotifications:
+    type: boolean
+    description: Enable/Disable the support for power state change callbacks to
+                 nova.
+    default: true
 
 parameter_groups:
 - label: deprecated
@@ -523,6 +528,14 @@ outputs:
             ironic::neutron::project_domain_name: 'Default'
             ironic::neutron::region_name: {get_param: KeystoneRegion}
             ironic::neutron::dhcpv6_stateful_address_count: {get_param: IronicDhcpv6StatefulAddressCount}
+            ironic::nova::auth_url: {get_param: [EndpointMap, KeystoneInternal, uri_no_suffix]}
+            ironic::nova::username: 'ironic'
+            ironic::nova::password: {get_param: IronicPassword}
+            ironic::nova::project_name: 'service'
+            ironic::nova::user_domain_name: 'Default'
+            ironic::nova::project_domain_name: 'Default'
+            ironic::nova::region_name: {get_param: KeystoneRegion}
+            ironic::nova::send_power_notifications: {get_param: IronicEnableNovaPowerNotifications}
             ironic::service_catalog::auth_url: {get_param: [EndpointMap, KeystoneInternal, uri_no_suffix]}
             ironic::service_catalog::username: 'ironic'
             ironic::service_catalog::password: {get_param: IronicPassword}

environments/undercloud.yaml

@@ -89,6 +89,7 @@ parameter_defaults:
   IronicConfigureSwiftTempUrlKey: false
   IronicProvisioningNetwork: 'ctlplane'
   IronicRescuingNetwork: 'ctlplane'
+  IronicEnableNovaPowerNotifications: false
   NeutronServicePlugins: router,segments
   NeutronMechanismDrivers: ['openvswitch', 'baremetal']
   NeutronNetworkVLANRanges: 'physnet1:1000:2999'

releasenotes/notes/ironic_send_power_notifications-f9977e3179b68868.yaml

@@ -0,0 +1,15 @@
+---
+features:
+  - |
+    Added paramter ``IronicEnableNovaPowerNotifications`` (defaults to:
+    ``true``). The parameter controls the ``[nova]/send_power_notifications``
+    option in ironic.conf which is used to enable/disable the power state
+    change callbacks to nova.
+fixes:
+  - |
+    The undercloud now disables ``[nova]/send_power_notifications`` in the
+    ironic service. This fixes an issue where ironic-conductor on the
+    undercloud would try to report power state changes to nova and fail
+    because nova service is not runnint on the undercloud. See bug:
+    `2000308 <https://bugs.launchpad.net/tripleo/+bug/2000308>`_.
+