Ensure cinder-backup-related variables are defined
This commit simply updates ceph_client/tasks/ceph_auth.yml to ensure
that cinder_service_backup_program_enabled and
cinder_service_backup_driver are defined before trying to evaluate
them. On a normal deployment, this is not an issue but when running
os-nova-install.yml on an AIO where ceph support is enabled, this
variable evaluation fails as the AIO host is part of both cinder_backup
and nova_compute groups, but does not have access to these variable
defaults in os_cinder/defaults/main.yml since we're operating on the
os_nova role.
An alternate solution would be to add these variable defaults to
group_vars, however as the os_nova role doesn't actually need to use
them it's probably cleaner to just check if they're defined.
Change-Id: I2c35b14f42b2c452aa68a611ab68c612db1b07a4
Closes-Bug: #1493799
(cherry picked from commit 466de05c67fa4ea071909a20872c3876711a5c23)
Reviewed: https:/ /review. openstack. org/222610 /git.openstack. org/cgit/ openstack/ openstack- ansible/ commit/ ?id=ba4661296ce 3f9703748c6a284 3cd7ff63712db4
Committed: https:/
Submitter: Jenkins
Branch: kilo
commit ba4661296ce3f97 03748c6a2843cd7 ff63712db4
Author: Matt Thompson <email address hidden>
Date: Wed Sep 9 12:07:50 2015 +0100
Ensure cinder- backup- related variables are defined
This commit simply updates ceph_client/ tasks/ceph_ auth.yml to ensure service_ backup_ program_ enabled and service_ backup_ driver are defined before trying to evaluate nova-install. yml on an AIO where ceph support is enabled, this defaults/ main.yml since we're operating on the
that cinder_
cinder_
them. On a normal deployment, this is not an issue but when running
os-
variable evaluation fails as the AIO host is part of both cinder_backup
and nova_compute groups, but does not have access to these variable
defaults in os_cinder/
os_nova role.
An alternate solution would be to add these variable defaults to
group_vars, however as the os_nova role doesn't actually need to use
them it's probably cleaner to just check if they're defined.
Change-Id: I2c35b14f42b2c4 52aa68a611ab68c 612db1b07a4 071909a20872c38 76711a5c23)
Closes-Bug: #1493799
(cherry picked from commit 466de05c67fa4ea