Clean up a few FIXME

Change-Id: I507efdf3e771789de8cae5e29bc312b4d27f6b25
glance: remove useless comments, they were implemented.
ironic: remove the useless condition
rabbitmq: remove useless comment
zaqar: remove useless parameter, call class directly

manifests/profile/base/glance/api.pp

@@ -193,11 +193,9 @@ class tripleo::profile::base::glance::api (
     $http_store = ['http']
     $glance_store = concat($http_store, $backend_store)
 
-    # TODO: notifications, scrubber, etc.
     include ::glance
     include ::glance::config
     include ::glance::api::logging
-    # TODO(jaosorior): Remove bind_host when we set it up conditionally in t-h-t
     class { '::glance::api':
       stores  => $glance_store,
       sync_db => $sync_db,

manifests/profile/base/ironic_inspector.pp

@@ -73,17 +73,9 @@ class tripleo::profile::base::ironic_inspector (
 
   if $step >= 4 or ($step >= 3 and $sync_db) {
     include ::ironic::inspector::cors
-    # TODO(hjensas): Once https://review.openstack.org/557465 is merged remove
-    # the condition.
-    if $inspection_subnets != [] {
-      class { '::ironic::inspector':
-        sync_db            => $sync_db,
-        dnsmasq_ip_subnets => $inspection_subnets_real,
-      }
-    } else {
-      class { '::ironic::inspector':
-        sync_db => $sync_db,
-      }
+    class { '::ironic::inspector':
+      sync_db            => $sync_db,
+      dnsmasq_ip_subnets => $inspection_subnets_real,
     }
 
     include ::ironic::inspector::pxe_filter

manifests/profile/base/rabbitmq.pp

@@ -113,7 +113,7 @@
 class tripleo::profile::base::rabbitmq (
   $certificate_specs             = {},
   $config_variables              = hiera('rabbitmq_config_variables'),
-  $enable_internal_tls           = undef,  # TODO(jaosorior): pass this via t-h-t
+  $enable_internal_tls           = undef,
   $environment                   = hiera('rabbitmq_environment'),
   $ssl_versions                  = undef,
   # lint:ignore:140chars

manifests/profile/base/zaqar.pp

@@ -119,9 +119,7 @@ class tripleo::profile::base::zaqar (
 
     # TODO (bcrochet): At some point, the transports should be split out to
     # separate services.
-    class { '::zaqar::server':
-      service_name => 'httpd', # TODO cleanup when passed by t-h-t.
-    }
+    include ::zaqar::server
     class { '::zaqar::wsgi::apache':
       ssl_cert => $tls_certfile,
       ssl_key  => $tls_keyfile,