remove cinder_volume_driver config option from globals.yml

[Steven Dake]

We don't need this configuration option. There are two types of config - ceph or lvm. We are not templating further support for cinder drivers. If people want more cinder drivers they can override the configuration to their liking.

Since we are introducing ceph, we can use the "enable_ceph" flag to determine if cinder should use ceph or should use lvm.

Use:

If enable_ceph=yes
do ceph config
else
do lvm config

vs current code of

if cinder_volume_driver=lvm
do lvm config
elsif cinder_volume_driver=ceph
do ceph config
endif

IRC Discussion:
http://eavesdrop.openstack.org/irclogs/%23kolla/%23kolla.2015-08-17.log.html#t2015-08-17T16:32:40

[Steven Dake (sdake)]

critical because must be fixed before liberty release. config file is in lockdown :)

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/213840

[OpenStack Infra (hudson-openstack)]

Change abandoned by Ryan Hallisey (<email address hidden>) on branch: master
Review: https://review.openstack.org/213840
Reason: sounds good

[Sam Yaple (s8m)]

The patch for this was abandoned. We have since moved virtually all of these options to group_vars/all.yml so this can be closed.

[Steven Dake (sdake)]

Sam this can still be addressed, just in group_vars/all.yml. Placing bug back in triaged state.

[Sam Yaple (s8m)]

We aren't removing this option though. It will still exist. Just because you have ceph doesn't mean you _will_ back cinder by it.

I am not sure what you want to track or accomplish by keeping this open, but there is nothing more to do here. Please update the bug with what you would like to see happen.