Bug 1311371

Summary: [RFE] Enable puppet to modify 'default_floating_pool' used by nova api
Product: Red Hat OpenStack Reporter: Jaison Raju <jraju>
Component: rhosp-directorAssignee: Angus Thomas <athomas>
Status: CLOSED CURRENTRELEASE QA Contact: Arik Chernetsky <achernet>
Severity: medium Docs Contact:
Priority: medium    
Version: 7.0 (Kilo)CC: athomas, dbecker, jcoufal, jraju, mburns, morazi, rhel-osp-director-maint, sgordon, srevivo
Target Milestone: ---Keywords: FutureFeature
Target Release: ---Flags: jraju: needinfo? (athomas)
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-01-24 21:24:41 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:

Description Jaison Raju 2016-02-24 05:22:36 UTC
1. Proposed title of this feature request  
      Enable puppet to modify 'default_floating_pool' used by nova api .

    3. What is the nature and description of the request?  
default_floating_pool is an parameter used by nova api to talk to
neutron server and associate floating IP to servers from the public
network. Upstream default is 'nova', let's keep it but at least this
patch allows to change the parameter.
      
    4. Why does the customer need this? (List the business requirements here)
      Need to non default value (nova) for this variable .
      Causing few tempest tests to fail with this ability .
      
    7. Is there already an existing RFE upstream or in Red Hat Bugzilla?  
      
https://github.com/openstack/puppet-nova/commit/83ea9464a5c99733521541095104fa34d6ee2614

Comment 3 Emilien Macchi 2016-02-24 16:14:07 UTC
There is no bug in puppet-nova, since we already provide the interface to configure default_floating_pool in ::nova::api class, like you mentioned with the commit.

Which installer are you using?

Comment 4 Jaison Raju 2016-03-25 04:37:23 UTC
(In reply to Emilien Macchi from comment #3)
> There is no bug in puppet-nova, since we already provide the interface to
> configure default_floating_pool in ::nova::api class, like you mentioned
> with the commit.
I am sorry , you are right :
#  grep -r default_floating /usr/share/openstack-puppet/
/usr/share/openstack-puppet/modules/nova/manifests/api.pp:# [*default_floating_pool*]
/usr/share/openstack-puppet/modules/nova/manifests/api.pp:  $default_floating_pool     = 'nova',
/usr/share/openstack-puppet/modules/nova/manifests/api.pp:    'DEFAULT/default_floating_pool':     value => $default_floating_pool;
/usr/share/openstack-puppet/modules/nova/CHANGELOG.md:- api: add default_floating_pool parameter

But not sure why , when i check the src rpm files , i couldn't find it .
$ grep floating openstack-puppet-modules-2015.1.8-51.el7ost/SOURCES/openstack-puppet-modules-2015.1.8/nova/manifests/api.pp
[No Results]


> 
> Which installer are you using?

I am checking with customer now . Customer may have made the same mistake as I did .

Comment 7 Mike Burns 2016-04-07 21:11:06 UTC
This bug did not make the OSP 8.0 release.  It is being deferred to OSP 10.