openstack
/
openstack-ansible
Code Issues Proposed changes

Fix cinder_service_region undefined 

Due to the simplification of group vars [1], we've removed the
definition of cinder_service_region for magnum.

The magnum role has been simplified in a separate commit to remove the
dependency on this weak linki (scoping only inside its role),
still allowing it to be overriden by a deployer. (See dependant commit).

With that included, we only needed to wire the magnum_service_region
in group vars, so we wire the magnum_service_region inside the magnum
group to the standard "service_region" variable, which puts
everything at the same region, like all the other services are doing,
and also standardizes the configuration.

[1]: https://review.openstack.org/#/c/504804/18/group_vars/all/cinder.yml

Depends-On: Id25c2d344859adfd09108b759d55f11ae83f97a9
Change-Id: Ibda9da1feb319a4ceaae1d470d4e1ea53755c378
(cherry picked from commit 8fdf4331ff)
This commit is contained in:
Jean-Philippe Evrard 2017-09-27 09:47:30 +01:00 committed by Mohammed Naser
parent 3f211f640b
commit a088b11738
1 changed files with 1 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
group_vars/magnum_all.yml
View File

@ -16,6 +16,7 @@
magnum_galera_user: magnum
magnum_galera_database_name: magnum_service
magnum_galera_address: "{{ galera_address }}"
magnum_service_region: "{{ service_region }}"
magnum_rabbitmq_userid: magnum
magnum_rabbitmq_vhost: /magnum