os_region_name settings deprecation broke IHA

Bug #1798560 reported by Michele Baldessari
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
Fix Released
High
Michele Baldessari

Bug Description

Via change f2e72352b1376ce719614e9cad4e4c71a3f9c3d8 we did the following:
diff --git a/puppet/services/nova-base.yaml b/puppet/services/nova-base.yaml
index f12b0d816dea..3e43b8cf7477 100644
--- a/puppet/services/nova-base.yaml
+++ b/puppet/services/nova-base.yaml
@@ -260,7 +260,7 @@ outputs:
           nova::placement::project_name: 'service'
           nova::placement::password: {get_param: NovaPassword}
           nova::placement::auth_url: {get_param: [EndpointMap, KeystoneInternal, uri_no_suffix]}
- nova::placement::os_region_name: {get_param: KeystoneRegion}
+ nova::placement::region_name: {get_param: KeystoneRegion}
           nova::placement::os_interface: {get_param: NovaPlacementAPIInterface}
           nova::database_connection:
             make_url:

But in the IHA script we looked for the os_region_name config key in the placemenet ini section of nova.conf. We need to adapt that script as well

Changed in tripleo:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tripleo-heat-templates (master)

Reviewed: https://review.openstack.org/611551
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=9235e30c0819d1283a55029f73b4d40fbe6625fe
Submitter: Zuul
Branch: master

commit 9235e30c0819d1283a55029f73b4d40fbe6625fe
Author: Michele Baldessari <email address hidden>
Date: Thu Oct 18 11:19:09 2018 +0200

    Fix IHA with the new region_name key

    ia change f2e72352b1376ce719614e9cad4e4c71a3f9c3d8 we did the following
    in nova-base.yaml:
    - nova::placement::os_region_name: {get_param: KeystoneRegion}
    + nova::placement::region_name: {get_param: KeystoneRegion}

    But in the IHA script we looked for the os_region_name config key in the
    placemenet ini section of nova.conf, which is now missing.
    We need to adapt that script as well. Without this fix we'll error
    out like this:
    Oct 16 09:08:12 overcloud-novacomputeiha-0 dockerd-current[14673]: File "/var/lib/nova/instanceha/check-run-nova-compute", line 147, in create_nova_connection
    Oct 16 09:08:12 overcloud-novacomputeiha-0 dockerd-current[14673]: region_name=options["os_region_name"][0],
    Oct 16 09:08:12 overcloud-novacomputeiha-0 dockerd-current[14673]: KeyError: 'os_region_name'

    Closes-Bug: #1798560

    Change-Id: I8906145955ab6c444efdfa73beca073a62c26e26

Changed in tripleo:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tripleo-heat-templates (stable/rocky)

Fix proposed to branch: stable/rocky
Review: https://review.openstack.org/613258

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on tripleo-heat-templates (stable/rocky)

Change abandoned by Juan Antonio Osorio Robles (<email address hidden>) on branch: stable/rocky
Review: https://review.openstack.org/613258
Reason: clearing up the gate to free up resources.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tripleo-heat-templates (stable/rocky)

Reviewed: https://review.openstack.org/613258
Committed: https://git.openstack.org/cgit/openstack/tripleo-heat-templates/commit/?id=aab90490b705463004b8cfe644af934fff76da21
Submitter: Zuul
Branch: stable/rocky

commit aab90490b705463004b8cfe644af934fff76da21
Author: Michele Baldessari <email address hidden>
Date: Thu Oct 18 11:19:09 2018 +0200

    Fix IHA with the new region_name key

    ia change f2e72352b1376ce719614e9cad4e4c71a3f9c3d8 we did the following
    in nova-base.yaml:
    - nova::placement::os_region_name: {get_param: KeystoneRegion}
    + nova::placement::region_name: {get_param: KeystoneRegion}

    But in the IHA script we looked for the os_region_name config key in the
    placemenet ini section of nova.conf, which is now missing.
    We need to adapt that script as well. Without this fix we'll error
    out like this:
    Oct 16 09:08:12 overcloud-novacomputeiha-0 dockerd-current[14673]: File "/var/lib/nova/instanceha/check-run-nova-compute", line 147, in create_nova_connection
    Oct 16 09:08:12 overcloud-novacomputeiha-0 dockerd-current[14673]: region_name=options["os_region_name"][0],
    Oct 16 09:08:12 overcloud-novacomputeiha-0 dockerd-current[14673]: KeyError: 'os_region_name'

    Closes-Bug: #1798560

    Change-Id: I8906145955ab6c444efdfa73beca073a62c26e26
    (cherry picked from commit 9235e30c0819d1283a55029f73b4d40fbe6625fe)

tags: added: in-stable-rocky
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tripleo-heat-templates 10.1.0

This issue was fixed in the openstack/tripleo-heat-templates 10.1.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tripleo-heat-templates 9.1.0

This issue was fixed in the openstack/tripleo-heat-templates 9.1.0 release.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.