BZ #1033243 - False Positive Warning when executing foreman_client.sh

https://bugzilla.redhat.com/show_bug.cgi?id=1033243
Patch adds text explaining 'no node definition' error to the user.
BZ #1033249 - Add overrides for keep_ip and bridge name

https://bugzilla.redhat.com/show_bug.cgi?id=1033249
Merge pull request #58 from jguiditta/bz1033243_client_output

BZ #1033243 - False Positive Warning when executing foreman_client.sh
Initial glusterfs
Added gluster readme
remove duplicate variable that caused error
Split cinder_storage class

The cinder_storage class served two different purposes:

* mounting of Gluster volumes into controller,

* providing a LVM block storage backend accessible through iSCSI.

Mounting of Gluster volumes has moved into cinder_controller, and the
LVM block storage backend has moved into
storage_backend::lvm_cinder. The cinder_storage class has been
removed.
Host groups: update LVM storage, add Gluster

Host group for LVM storage has been updated to reflect the class move
and a new host group for Gluster storage backend (general for Cinder,
Glance, Swift in the future) has been added.
Cascade parameters from storage_backend::gluster

We need to pass parameters needed for storage_backend::gluster::cinder
and storage_backend::gluster::glance into storage_backend::gluster (top
class for Gluster Storage host group), because we want Foreman to
override them, instead of inheriting values from quickstack::params.
params cinder_gluster_volume and glance_gluster_volume should also be overridable
remove quickstack::neutron::network_public declaration from controller
use friendly str2bool
Add cinder::volume to cinder_controller when using Gluster

Having cinder::volume there is necessary when using the Gluster
backend. It used to be included in the controller after Gilles'
commits, but i erred when refactoring and removed it, so i'm putting
it back.
Honor parameter values for naming gluster volumes

$cinder_gluster_volume and $glance_gluster_volume parameters used to
affect only controller, but we should also use them in Gluster
backend. We need them synced if we want mounting on controller to
work.
Current cinder_controller also for Nova Network

Update parameter passing in nova_network::controller.
Sync defaults in seeds.rb with defaults in params.pp

Update the default names of Gluster volumes to match the ones in
params.pp (users can still override them if they want).

Not all defaults are synced. E.g. i think it doesn't make much sense
to provide default IP addresses of storage nodes, because the chance
that user could use these defaults is almost zero.
Initial glusterfs
Split cinder_storage class

The cinder_storage class served two different purposes:

* mounting of Gluster volumes into controller,

* providing a LVM block storage backend accessible through iSCSI.

Mounting of Gluster volumes has moved into cinder_controller, and the
LVM block storage backend has moved into
storage_backend::lvm_cinder. The cinder_storage class has been
removed.
Cascade parameters from storage_backend::gluster

We need to pass parameters needed for storage_backend::gluster::cinder
and storage_backend::gluster::glance into storage_backend::gluster (top
class for Gluster Storage host group), because we want Foreman to
override them, instead of inheriting values from quickstack::params.
remove quickstack::neutron::network_public declaration from controller