Compute and controller syntax fixes

The missing colons after class names prevented the classes from
getting imported into foreman.
Top-down parameter passing from host group classes

We need to pass parameters from host group classes
(quickstack::controller and quickstack::compute) into "resourced"
classes inside them, so that parameters can be overriden by
Foreman. (Otherwise, hardcoded values from params.pp would be used.)

Parameter lists are now sorted alphabetically for easier orientation.
Remove unused admin_password from quickstack::neutron::compute
Fix neutron keystone auth config

Neutron::keystone::auth was resourced twice when $neutron == 'true',
and this caused puppet error. I first tried to remove it from
quickstack::controller and keep it in quickstack::neutron::controller,
but the openstack::keystone class seems to require
neutron_user_password even when neutron is disabled, so we'll better
configure it from quickstack::controller and just use the `neutron`
boolean parameter to enable or disable creating keystone entries for
neutron.

Also, wrap $neutron variable reading into str2bool as it is in other
places.
Merge pull request #64 from cwolferh/rebase_pr_63_but_default_neutron_false

 Rebase PR 63