Bug 1698712

Summary: availability_zone class is not included in scheduler.pp
Product: Red Hat OpenStack Reporter: Keigo Noha <knoha>
Component: puppet-novaAssignee: Rajesh Tailor <ratailor>
Status: CLOSED ERRATA QA Contact: OSP DFG:Compute <osp-dfg-compute>
Severity: medium Docs Contact:
Priority: medium    
Version: 13.0 (Queens)CC: jjoyce, jschluet, mschuppe, nlevinki, ratailor, slinaber, tvignaud
Target Milestone: z7Keywords: Triaged, ZStream
Target Release: 13.0 (Queens)   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: puppet-nova-12.4.0-19.el7ost Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1706711 1706714 (view as bug list) Environment:
Last Closed: 2019-07-10 13:05:11 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1706711, 1706714    

Description Keigo Noha 2019-04-11 04:36:13 UTC
Description of problem:
nova-scheduler distinguish availability zone in metadata or CONF.default_availability_zone. However, scheduler.pp doesn't contain nova::availability_zone class.
Because of that, nova.conf for nova-scheduler in controller needs to be updated manually.
It should be included in scheduler.pp.

This trigger a failure of instance creation with volume when a user defines an availability zone in Cinder and cross_az_attach option in nova.conf is false.

Version-Release number of selected component (if applicable):
puppet-nova-12.4.0-17

How reproducible:
Always

Steps to Reproduce:
1. Configure availability_zone in cinder and nova through director.
~~~
parameter_defaults:
  ExtraConfig:
    nova::availability_zone::default_availability_zone: "AZ_TEST"
    cinder::storage_availability_zone: "AZ_TEST"
~~~
2. After deployment, try to launch the instance with volume.
3.

Actual results:
The instance creation failed.

Expected results:
The instance creation succeed.

Additional info:
The issue is filed as https://bugs.launchpad.net/puppet-nova/+bug/1824273
And Gerrit is https://review.openstack.org/#/c/651690/

Comment 1 Keigo Noha 2019-04-19 00:14:54 UTC
Hi Rajesh,

The gerrit for master branch was merged status. Now, the fix is being proposed to stable/rocky and stable/queens branch. Would you please follow-up those gerrits in upstream?

Kind Regards,
Keigo Noha

Comment 2 Rajesh Tailor 2019-04-22 05:32:48 UTC
Hi Keigo,

Thanks for the update.
I will backport those patches downstream in OSP14 and OSP13, as soon as they are merged upstream.

Comment 3 Keigo Noha 2019-05-15 01:46:34 UTC
Hi Rajesh,

All patch into stable/queens and stable/rocky was merged. Would you please backport them into RHOSP internal gerrit?

Kind Regards,
Keigo Noha

Comment 5 Keigo Noha 2019-05-22 01:09:06 UTC
Hi Rajesh,

Thank you for your work in upstream.
In upstream, Tobias commented that the fix should not be done in puppet-nova.
If the fix should be done in puppet-tripleo, I'm happy to revert a patch then put it into puppet-tripleo.
Martin shared his opinion to Tobias's comment. But if I don't align to the contributor's guide or the rules in puppet-nova, I'd like to fix it.

I'd like to hear your opinion to it.

Kind Regards,
Keigo Noha

Comment 15 errata-xmlrpc 2019-07-10 13:05:11 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2019:1738