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')
Fix proposed to branch: master /review. openstack. org/258155
Review: https:/