openstack/openstack-ansible
Code Issues Proposed changes

Provide Nova & Cinder the Glance API Servers

Browse Source 
Currently the Nova and Cinder roles require the Glance API server
list, but use variables outside of their namespace.

This is not good practise. This sets up namespaced group_vars for
use by both roles.

The Cinder role already expects the var [1], but a follow-on patch
will be submitted to the Nova role to also adjust its pattern.

[1] https://review.openstack.org/290856

Change-Id: I31b06aad686fcc32d55c78147e3d5d099b8cc5b9
This commit is contained in:
Jesse Pretorius
2016-03-31 14:23:00 +01:00
committed by Jesse Pretorius (odyssey4me)
parent a21cb794a7
commit 458a67535a
1 changed files with 2 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
playbooks/inventory/group_vars/hosts.yml
View File

@@ -146,6 +146,7 @@ nova_dhcp_domain: "{{ dhcp_domain }}"
nova_service_in_ldap: "{{ service_ldap_backend_enabled }}"
nova_rabbitmq_userid: nova
nova_rabbitmq_vhost: /nova
nova_glance_api_servers: "{{ glance_api_servers }}"
## Neutron
@@ -252,6 +253,7 @@ cinder_service_region: "{{ service_region }}"
cinder_service_in_ldap: "{{ service_ldap_backend_enabled }}"
cinder_rabbitmq_userid: cinder
cinder_rabbitmq_vhost: /cinder
cinder_glance_api_servers: "{{ glance_api_servers }}"
cinder_glance_api_version: "{{ (cinder_backends_rbd_inuse|bool and glance_default_store == 'rbd') | ternary('2','1') }}"