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.
Reviewed: https:/ /review. openstack. org/221705 /git.openstack. org/cgit/ stackforge/ os-ansible- deployment/ commit/ ?id=466de05c67f a4ea071909a2087 2c3876711a5c23
Committed: https:/
Submitter: Jenkins
Branch: master
commit 466de05c67fa4ea 071909a20872c38 76711a5c23
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
Closes-Bug: #1493799