openstack/devstack-plugin-ceph
Code Issues Proposed changes

Only import libvirt secret when nova is enabled

Browse Source 
This fixes a small logical error, we only want to set a libvirt secret
if nova is enabled, not both cinder and nova.  This prevents a
deployment with cinder but not nova from being successful.

Change-Id: Ief83fe661cf1b2099af8131a7a206ffca3458dc7
This commit is contained in:
Jon Bernard 2018-03-20 12:40:58 -04:00
parent 41d575b2dc
commit c5c0adcc60
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
devstack/plugin.sh
View File

@ -53,8 +53,7 @@ elif [[ "$1" == "stack" && "$2" == "post-config" ]]; then
echo_summary "Configuring Cinder for Ceph" echo_summary "Configuring Cinder for Ceph"
configure_ceph_cinder configure_ceph_cinder
fi fi
if is_ceph_enabled_for_service cinder || \ if is_ceph_enabled_for_service nova; then
is_ceph_enabled_for_service nova; then
# NOTE (leseb): the part below is a requirement # NOTE (leseb): the part below is a requirement
# to attach Ceph block devices # to attach Ceph block devices
echo_summary "Configuring libvirt secret" echo_summary "Configuring libvirt secret"