puppet: correct cinder enabled_backends format
authorDan Prince <dprince@redhat.com>
Sun, 29 Mar 2015 01:46:54 +0000 (21:46 -0400)
committerDan Prince <dprince@redhat.com>
Sun, 29 Mar 2015 01:51:03 +0000 (21:51 -0400)
commit2c156c1c1370d409e0b58858d78c3c6e26cda6cd
treea0ab23c625d81d659149db0f951526835bfa8a55
parentfed9d001ccc92a6d3392cb959416596b6df83d99
puppet: correct cinder enabled_backends format

This resolves a formatting issue with the Cinder enabled_backends
config file setting. Previously we would potentially construct
an array with an undef value at the end if iscsi was enabled
but ceph was not (this is the case for our current CI job).
When an array formatted like ['tripleo_iscsi', undef] is then
passed to join() in puppet-cinder to construct a string it leaves
us with an extra ',' on the end of the string. This causes
problems in that cinder-volume loads an extra (system default)
cinder volume process which is not expected.

Because Fedora uses LIO as a default it was causing about half
of our CI runs to fail if the tgtadm cinder-volume process
wasn't being chosen by the scheduler.

Closes-bug: #1437708

Change-Id: I3383012cb43792f334fdf789dc13147a3cb5ad63
puppet/manifests/overcloud_controller.pp