remove hardcoded network roles from manifests

Bug #1505604 reported by Vasyl Saienko
20
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Medium
Maksim Malchuk

Bug Description

At the moment some manifests contain hardcoded network roles:

$ grep "get_node_to_ipaddr_map_by_network_role(get_nodes_hash_by_roles" * -R
deployment/puppet/osnailyfacter/modular/openstack-haproxy/openstack-haproxy-heat.pp:$heat_address_map = get_node_to_ipaddr_map_by_network_role(get_nodes_hash_by_roles($network_metadata, hiera('heat_roles')), 'heat/api')
deployment/puppet/osnailyfacter/modular/openstack-haproxy/openstack-haproxy-sahara.pp:$sahara_address_map = get_node_to_ipaddr_map_by_network_role(get_nodes_hash_by_roles($network_metadata, hiera('sahara_roles')), 'sahara/api')
deployment/puppet/osnailyfacter/modular/openstack-haproxy/openstack-haproxy-keystone.pp:$keystones_address_map = get_node_to_ipaddr_map_by_network_role(get_nodes_hash_by_roles($network_metadata, ['primary-controller', 'controller']), 'keystone/api')
deployment/puppet/osnailyfacter/modular/openstack-haproxy/openstack-haproxy-murano.pp:$murano_address_map = get_node_to_ipaddr_map_by_network_role(get_nodes_hash_by_roles($network_metadata, hiera('murano_roles')), 'murano/api')
deployment/puppet/osnailyfacter/modular/openstack-haproxy/openstack-haproxy-glance.pp:$glances_address_map = get_node_to_ipaddr_map_by_network_role(get_nodes_hash_by_roles($network_metadata, ['primary-controller', 'controller']), 'glance/api')

they should be replaced with globals like:
deployment/puppet/osnailyfacter/modular/openstack-haproxy/openstack-haproxy-cinder.pp:$cinder_address_map = get_node_to_ipaddr_map_by_network_role(hiera_hash('cinder_nodes'), 'cinder/api')

Vasyl Saienko (vsaienko)
Changed in fuel:
milestone: none → 8.0
Changed in fuel:
assignee: nobody → Fuel Library Team (fuel-library)
tags: added: feature-advanced-networking tech-debt
Changed in fuel:
importance: Undecided → Medium
status: New → Confirmed
Dmitry Pyzhov (dpyzhov)
tags: added: area-library
Changed in fuel:
assignee: Fuel Library Team (fuel-library) → Maksim Malchuk (mmalchuk)
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (master)

Fix proposed to branch: master
Review: https://review.openstack.org/258155

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/258155
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=aacacc9bf9736c971d62bff872e197df39e33325
Submitter: Jenkins
Branch: master

commit aacacc9bf9736c971d62bff872e197df39e33325
Author: Maksim Malchuk <email address hidden>
Date: Wed Dec 30 18:59:50 2015 +0300

    Remove hardcoded network roles from HAproxy manifests

    This commit removed some hardcoded network roles from HAproxy manifests
    and replaces them with globals. Also it fixes hiera override functionality
    for some globals.

    Change-Id: Ia6006b19f366cec8ac686e19e7b921e95051c1ac
    Closes-Bug: #1505604

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Timur Nurlygayanov (tnurlygayanov) wrote :

Looks like it works fine since we don't have any issues with network interfaces.
Status changed to Fix Released for MOS 8.0

Changed in fuel:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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