disable mysql_account_security

for some reason, I got the following failure:

err: Failed to apply catalog: Parameter name failed: Invalid database user root@ at /etc/puppet/modules/mysql/manifests/server/account_security.pp:8

I did not take the time to figure out what it meant
and instead just commented out this code to
see if it would go away. clearly, this commit
should never be merged, and we just identify and
resolve the real problem,
remove extra $

this commit simply removes an extra $ that was
entered by mistake. The extra dollar sign is ignored
by Puppet, but it is obviously invalid syntax.
do not create nova volumes endpoints

Since the openstack module no longer supports nova
volumes, this commit sets the cinder flag of
the nova::keystone::auth class which ensures that
nova-volume endpoints will not be configured.

This commit also refactors the unit tests a bit to
use required_params instead of default_params.
provide a default for db_host

In general, I prefer to provide reasonable defaults
when they exist.

This commit provides 127.0.0.1 as the default for
db_host.
It also rearranges the class params so that only
the required ones come first.
Configure db_host

TODO - Daneyon, why exactly is this required?

Previously an option called db_host existed in
the db/mysql class, but it wasn't actually used.

This commit updates the databases created to
specify what host the db resides on.

This commit also adds some basic unit tests for
openstack::db::mysql.
refactor quantum configuration

TODO - update unit tests

add special openstack::quantum manifest that can
be used to configure various quantum roles.

nova::controller changes:

- remove quantum code from nova::controller
(b/c it is not part of that role)
- specify that the private and public interface
must be set when using nova-networking in the
nova::controller (this is b/c they are no longer
required to be passed into the openstack::controller
b/c quantum does not need them)

openstack::controller changes:
* configure quantum when configured to do so
update cinder config in controller

TODO - add tests, update openstack::all

update the cinder configuration in the controller
to use the new openstack::cinder::controller class.
refactor db deps

I really want to see all of these nasty deps
removes, but it will take some time to
understand why they are even there in the first
place.
glance refactor

this refactor of glance does three things:

* makes the backend pluggable and adds support
for a swift backend.
* removes repetitive glance_* prefixes from
variables.
* updates the openstack::controller class to use
these new glance params
allow configuration of keystone bind_host
allow configuration of keystone_host

in order to allow keystone to be on a different node
for HA.
align arrows
move verbose under general params
refactor of cinder manifests

This commit refactors the cinder manifests to make
them more consistent with other manifests:

* removed default value for keystone_password
* removed the prefix cinder_ from class parameters
(it is redundant)
* set package_ensure defaults to present
* removed extra package_ensure parameter
* changed $iscsi_enabled to $volume_driver (this
change is being done in anticipation of needing
additional volume drivers)
* add test coverage
* simple updates to .gitignore
* refactor sql_connection to be composed of multiple class paramters
for consistency.

It also delete the openstack::cinder class b/c
I could not seem to declare ::cinder while it
existed.

Change-Id: I4a7b49d95957675be82c3b77958ae9d0c47eb4fa
align some arrows