Fix parameters and headers inconsistency in the puppet manifests.
authorCarlos Camacho <ccamacho@redhat.com>
Fri, 5 Aug 2016 09:54:25 +0000 (11:54 +0200)
committerCarlos Camacho <ccamacho@redhat.com>
Mon, 8 Aug 2016 20:44:01 +0000 (22:44 +0200)
As we are staring to manually check overcloud services
the first step is to check that the puppet profiles
are all aligned.

Changes applied:

  No logic added or removed in this submission.

  Removed unused parameters.

  Align header comments structure.

  All profiles parameters sorted following:
  "Mandatory params first sorted alphabetically
  then optional params sorted alphabetically."

Note: Following submissions will check pacemaker,
cinder, mistral and redis services in the base profiles
as some of them has the $pacemaker_master parameter
defaulted to true.

Change-Id: I2f91c3f6baa33f74b5625789eec83233179a9655

129 files changed:
manifests/profile/base/ceilometer/agent/central.pp
manifests/profile/base/ceilometer/agent/compute.pp
manifests/profile/base/ceilometer/agent/notification.pp
manifests/profile/base/ceilometer/api.pp
manifests/profile/base/ceilometer/collector.pp
manifests/profile/base/ceilometer/expirer.pp
manifests/profile/base/ceph/client.pp
manifests/profile/base/ceph/mon.pp
manifests/profile/base/ceph/osd.pp
manifests/profile/base/cinder.pp
manifests/profile/base/cinder/api.pp
manifests/profile/base/cinder/scheduler.pp
manifests/profile/base/cinder/volume.pp
manifests/profile/base/cinder/volume/dellsc.pp
manifests/profile/base/cinder/volume/eqlx.pp
manifests/profile/base/cinder/volume/iscsi.pp
manifests/profile/base/cinder/volume/netapp.pp
manifests/profile/base/cinder/volume/nfs.pp
manifests/profile/base/cinder/volume/rbd.pp
manifests/profile/base/database/mongodb.pp
manifests/profile/base/database/mongodbcommon.pp
manifests/profile/base/database/mysql.pp
manifests/profile/base/database/redis.pp
manifests/profile/base/glance/api.pp
manifests/profile/base/glance/registry.pp
manifests/profile/base/gnocchi.pp
manifests/profile/base/gnocchi/api.pp
manifests/profile/base/gnocchi/metricd.pp
manifests/profile/base/gnocchi/statsd.pp
manifests/profile/base/haproxy.pp
manifests/profile/base/heat.pp
manifests/profile/base/heat/api.pp
manifests/profile/base/heat/api_cfn.pp
manifests/profile/base/heat/api_cloudwatch.pp
manifests/profile/base/heat/engine.pp
manifests/profile/base/ironic.pp
manifests/profile/base/ironic/api.pp
manifests/profile/base/ironic/conductor.pp
manifests/profile/base/keepalived.pp
manifests/profile/base/keystone.pp
manifests/profile/base/manila/api.pp
manifests/profile/base/manila/scheduler.pp
manifests/profile/base/manila/share.pp
manifests/profile/base/memcached.pp
manifests/profile/base/mistral.pp
manifests/profile/base/mistral/api.pp
manifests/profile/base/mistral/engine.pp
manifests/profile/base/mistral/executor.pp
manifests/profile/base/monitoring/fluentd.pp
manifests/profile/base/monitoring/rabbitmq.pp
manifests/profile/base/monitoring/sensu.pp
manifests/profile/base/monitoring/uchiwa.pp
manifests/profile/base/neutron/agents/midonet.pp
manifests/profile/base/neutron/agents/nuage.pp
manifests/profile/base/neutron/dhcp.pp
manifests/profile/base/neutron/l3.pp
manifests/profile/base/neutron/metadata.pp
manifests/profile/base/neutron/midonet.pp
manifests/profile/base/neutron/ovs.pp
manifests/profile/base/neutron/plugins/ml2.pp
manifests/profile/base/neutron/plugins/nuage.pp
manifests/profile/base/neutron/plugins/opencontrail.pp
manifests/profile/base/neutron/plugins/plumgrid.pp
manifests/profile/base/nova.pp
manifests/profile/base/nova/compute/ironic.pp
manifests/profile/base/nova/compute/libvirt.pp
manifests/profile/base/nova/conductor.pp
manifests/profile/base/nova/consoleauth.pp
manifests/profile/base/nova/libvirt.pp
manifests/profile/base/nova/scheduler.pp
manifests/profile/base/nova/vncproxy.pp
manifests/profile/base/pacemaker.pp
manifests/profile/base/rabbitmq.pp
manifests/profile/base/sahara/api.pp
manifests/profile/base/sahara/engine.pp
manifests/profile/base/snmp.pp
manifests/profile/base/swift/add_devices.pp
manifests/profile/base/swift/proxy.pp
manifests/profile/base/swift/ringbuilder.pp
manifests/profile/base/swift/storage.pp
manifests/profile/base/trove/api.pp
manifests/profile/pacemaker/apache.pp
manifests/profile/pacemaker/ceilometer.pp
manifests/profile/pacemaker/ceilometer/agent/central.pp
manifests/profile/pacemaker/ceilometer/agent/notification.pp
manifests/profile/pacemaker/ceilometer/api.pp
manifests/profile/pacemaker/ceilometer/collector.pp
manifests/profile/pacemaker/cinder/api.pp
manifests/profile/pacemaker/cinder/scheduler.pp
manifests/profile/pacemaker/cinder/volume.pp
manifests/profile/pacemaker/core.pp
manifests/profile/pacemaker/database/mongodb.pp
manifests/profile/pacemaker/database/mysql.pp
manifests/profile/pacemaker/database/redis.pp
manifests/profile/pacemaker/glance.pp
manifests/profile/pacemaker/gnocchi.pp
manifests/profile/pacemaker/gnocchi/api.pp
manifests/profile/pacemaker/gnocchi/metricd.pp
manifests/profile/pacemaker/gnocchi/statsd.pp
manifests/profile/pacemaker/haproxy.pp
manifests/profile/pacemaker/heat.pp
manifests/profile/pacemaker/heat/api.pp
manifests/profile/pacemaker/heat/api_cfn.pp
manifests/profile/pacemaker/heat/api_cloudwatch.pp
manifests/profile/pacemaker/heat/engine.pp
manifests/profile/pacemaker/keystone.pp
manifests/profile/pacemaker/manila.pp
manifests/profile/pacemaker/memcached.pp
manifests/profile/pacemaker/neutron.pp
manifests/profile/pacemaker/neutron/dhcp.pp
manifests/profile/pacemaker/neutron/l3.pp
manifests/profile/pacemaker/neutron/metadata.pp
manifests/profile/pacemaker/neutron/midonet.pp
manifests/profile/pacemaker/neutron/ovs.pp
manifests/profile/pacemaker/neutron/plugins/ml2.pp
manifests/profile/pacemaker/neutron/plugins/nuage.pp
manifests/profile/pacemaker/neutron/plugins/opencontrail.pp
manifests/profile/pacemaker/neutron/plugins/plumgrid.pp
manifests/profile/pacemaker/neutron/server.pp
manifests/profile/pacemaker/nova.pp
manifests/profile/pacemaker/nova/api.pp
manifests/profile/pacemaker/nova/conductor.pp
manifests/profile/pacemaker/nova/consoleauth.pp
manifests/profile/pacemaker/nova/scheduler.pp
manifests/profile/pacemaker/nova/vncproxy.pp
manifests/profile/pacemaker/rabbitmq.pp
manifests/profile/pacemaker/sahara.pp
manifests/profile/pacemaker/sahara/api.pp
manifests/profile/pacemaker/sahara/engine.pp

index 02d6d1a..c91e610 100644 (file)
 #   for more details.
 #   Defaults to hiera('step')
 #
-
 class tripleo::profile::base::ceilometer::agent::central (
   $step = hiera('step'),
 ) {
-
   include ::tripleo::profile::base::ceilometer
 
   if $step >= 4 {
index 3a7aa50..749bc64 100644 (file)
 #   for more details.
 #   Defaults to hiera('step')
 #
-
 class tripleo::profile::base::ceilometer::agent::compute (
   $step = hiera('step'),
 ) {
-
   include ::tripleo::profile::base::ceilometer
 
   if $step >= 4 {
index 83a0234..7fe8e81 100644 (file)
 #   for more details.
 #   Defaults to hiera('step')
 #
-
 class tripleo::profile::base::ceilometer::agent::notification (
   $step = hiera('step'),
 ) {
-
   include ::tripleo::profile::base::ceilometer
 
   if $step >= 4 {
index e324c00..b300a91 100644 (file)
 #   for more details.
 #   Defaults to hiera('step')
 #
-
 class tripleo::profile::base::ceilometer::api (
   $step = hiera('step'),
 ) {
-
   include ::tripleo::profile::base::ceilometer
 
   if $step >= 4 {
index b0a986f..a8ca88b 100644 (file)
@@ -26,7 +26,7 @@
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
-
+#
 class tripleo::profile::base::ceilometer::collector (
   $bootstrap_node = hiera('bootstrap_nodeid', undef),
   $step           = hiera('step'),
index 43d8c26..0830307 100644 (file)
 #   for more details.
 #   Defaults to hiera('step')
 #
-
 class tripleo::profile::base::ceilometer::expirer (
   $step = hiera('step'),
 ) {
-
   include ::tripleo::profile::base::ceilometer
 
   if $step >= 4 {
index 851324a..53f09c2 100644 (file)
@@ -26,7 +26,6 @@
 class tripleo::profile::base::ceph::client (
   $step = hiera('step'),
 ) {
-
   include ::tripleo::profile::base::ceph
 
   if $step >= 2 {
index b211120..c0768b6 100644 (file)
@@ -37,7 +37,6 @@ class tripleo::profile::base::ceph::mon (
   $ceph_pools = {},
   $step       = hiera('step'),
 ) {
-
   include ::tripleo::profile::base::ceph
 
   if $step >= 2 {
index 9736656..6940bca 100644 (file)
@@ -31,7 +31,6 @@ class tripleo::profile::base::ceph::osd (
   $ceph_osd_selinux_permissive = false,
   $step                        = hiera('step'),
 ) {
-
   include ::tripleo::profile::base::ceph
 
   if $step >= 3 {
index 27dc277..8f252d4 100644 (file)
@@ -37,7 +37,6 @@ class tripleo::profile::base::cinder (
   $pacemaker_master       = true,
   $step                   = hiera('step'),
 ) {
-
   if $step >= 4 or ($step >= 3 and $pacemaker_master) {
     include ::cinder
     include ::cinder::config
index ad970f6..e15e112 100644 (file)
@@ -26,7 +26,7 @@
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
-
+#
 class tripleo::profile::base::cinder::api (
   $bootstrap_node = hiera('bootstrap_nodeid', undef),
   $step           = hiera('step'),
index 68f2813..4586929 100644 (file)
@@ -26,7 +26,6 @@
 class tripleo::profile::base::cinder::scheduler (
   $step = hiera('step'),
 ) {
-
   include ::tripleo::profile::base::cinder
 
   if $step >= 4 {
index 96cd06a..dfb034f 100644 (file)
@@ -61,7 +61,6 @@ class tripleo::profile::base::cinder::volume (
   $cinder_user_enabled_backends = hiera('cinder_user_enabled_backends', undef),
   $step                         = hiera('step'),
 ) {
-
   include ::tripleo::profile::base::cinder
 
   if $step >= 4 {
index 6f7922d..534bcb7 100644 (file)
@@ -31,7 +31,6 @@ class tripleo::profile::base::cinder::volume::dellsc (
   $backend_name = hiera('cinder::backend::dellsc_iscsi::volume_backend_name', 'tripleo_dellsc'),
   $step         = hiera('step'),
 ) {
-
   include ::tripleo::profile::base::cinder::volume
 
   if $step >= 4 {
index a18270d..2399459 100644 (file)
@@ -31,7 +31,6 @@ class tripleo::profile::base::cinder::volume::eqlx (
   $backend_name = hiera('cinder::backend::eqlx::volume_backend_name', 'tripleo_eqlx'),
   $step         = hiera('step'),
 ) {
-
   include ::tripleo::profile::base::cinder::volume
 
   if $step >= 4 {
index d857caa..d1d22bb 100644 (file)
 # [*cinder_iscsi_address*]
 #   The address where to bind the iscsi targets daemon
 #
+# [*backend_name*]
+#   (Optional) Name given to the Cinder backend stanza
+#   Defaults to 'tripleo_iscsi'
+#
 # [*cinder_iscsi_helper*]
 #   (Optional) The iscsi helper to use
 #   Defaults to 'tgtadm'
 #   (Optional) The size (in MB) of the LVM loopback volume
 #   Defaults to '10280'
 #
-# [*backend_name*]
-#   (Optional) Name given to the Cinder backend stanza
-#   Defaults to 'tripleo_iscsi'
-#
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #
 class tripleo::profile::base::cinder::volume::iscsi (
   $cinder_iscsi_address,
-  $cinder_iscsi_helper          = 'tgtadm',
-  $cinder_lvm_loop_device_size  = '10280',
-  $backend_name                 = hiera('cinder::backend::iscsi::volume_backend_name', 'tripleo_iscsi'),
-  $step                         = hiera('step'),
+  $backend_name                = hiera('cinder::backend::iscsi::volume_backend_name', 'tripleo_iscsi'),
+  $cinder_iscsi_helper         = 'tgtadm',
+  $cinder_lvm_loop_device_size = '10280',
+  $step                        = hiera('step'),
 ) {
-
   include ::tripleo::profile::base::cinder::volume
 
   if $step >= 4 {
index bae541f..fc652c9 100644 (file)
@@ -31,7 +31,6 @@ class tripleo::profile::base::cinder::volume::netapp (
   $backend_name = hiera('cinder::backend::netapp::volume_backend_name', 'tripleo_netapp'),
   $step         = hiera('step'),
 ) {
-
   include ::tripleo::profile::base::cinder::volume
 
   if $step >= 4 {
index a26c50e..7b1f1b9 100644 (file)
 #
 # === Parameters
 #
-# [*cinder_nfs_mount_options*]
-#   (Optional) List of mount options for the NFS share
-#   Defaults to ''
-#
 # [*cinder_nfs_servers*]
 #   List of NFS shares to mount
 #
 #   (Optional) Name given to the Cinder backend stanza
 #   Defaults to 'tripleo_nfs'
 #
+# [*cinder_nfs_mount_options*]
+#   (Optional) List of mount options for the NFS share
+#   Defaults to ''
+#
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
 #
 class tripleo::profile::base::cinder::volume::nfs (
-  $cinder_nfs_mount_options = '',
   $cinder_nfs_servers,
   $backend_name             = hiera('cinder::backend::nfs::volume_backend_name', 'tripleo_nfs'),
+  $cinder_nfs_mount_options = '',
   $step                     = hiera('step'),
 ) {
-
   include ::tripleo::profile::base::cinder::volume
 
   if $step >= 4 {
index 6c8341a..1246de8 100644 (file)
 #
 # === Parameters
 #
+# [*backend_name*]
+#   (Optional) Name given to the Cinder backend stanza
+#   Defaults to 'tripleo_ceph'
+#
 # [*cinder_rbd_backend_host*]
 #   (Optional) String to use as backend_host in the backend stanza
 #   Defaults to 'cinder::host'
 #   (Optional) The user name for the RBD client
 #   Defaults to 'openstack'
 #
-# [*backend_name*]
-#   (Optional) Name given to the Cinder backend stanza
-#   Defaults to 'tripleo_ceph'
-#
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
 #
 class tripleo::profile::base::cinder::volume::rbd (
+  $backend_name            = hiera('cinder::backend::rbd::volume_backend_name', 'tripleo_ceph'),
   $cinder_rbd_backend_host = hiera('cinder::host', 'hostgroup'),
   $cinder_rbd_pool_name    = 'volumes',
   $cinder_rbd_secret_uuid  = hiera('ceph::profile::params::fsid', undef),
   $cinder_rbd_user_name    = 'openstack',
-  $backend_name            = hiera('cinder::backend::rbd::volume_backend_name', 'tripleo_ceph'),
   $step                    = hiera('step'),
 ) {
-
   include ::tripleo::profile::base::cinder::volume
 
   if $step >= 4 {
index 4c53c9c..1a19bb6 100644 (file)
 #
 class tripleo::profile::base::database::mongodb (
   $mongodb_replset,
-  $bootstrap_node       = downcase(hiera('bootstrap_nodeid')),
-  $step                 = hiera('step'),
+  $bootstrap_node = downcase(hiera('bootstrap_nodeid')),
+  $step           = hiera('step'),
 ) {
-
   if $step >= 2 {
 
     include ::mongodb::globals
index 6530730..756ab85 100644 (file)
 #
 # === Parameters
 #
-# [*mongodb_node_ips*]
-#   List of The mongodb node ip addresses
-#
 # [*mongodb_ipv6_enabled*]
 #   A boolean value for mongodb server ipv6 is enabled or not
 #   Defaults to false
 #
+# [*mongodb_node_ips*]
+#   List of The mongodb node ip addresses
+#
 class tripleo::profile::base::database::mongodbcommon (
-  $mongodb_node_ips     = hiera('mongo_node_ips'),
   $mongodb_ipv6_enabled = false,
+  $mongodb_node_ips     = hiera('mongo_node_ips'),
 ) {
-
   $port = '27017'
 
   # NOTE(gfidente): the following vars are needed on all nodes so they
index 27df6e4..49c9df3 100644 (file)
 #
 # === Parameters
 #
-# [*step*]
-#   (Optional) The current step in deployment. See tripleo-heat-templates
-#   for more details.
-#   Defaults to hiera('step')
+# [*manage_resources*]
+#   (Optional) Whether or not manage root user, root my.cnf, and service.
+#   Defaults to true
 #
 # [*mysql_server_options*]
 #   (Optional) Extras options to deploy MySQL. Useful when deploying Galera cluster.
 #   Should be an hash.
 #   Defaults to {}
 #
-# [*manage_resources*]
-#   (Optional) Whether or not manage root user, root my.cnf, and service.
-#   Defaults to true
-#
 # [*remove_default_accounts*]
 #   (Optional) Whether or not remove default MySQL accounts.
 #   Defaults to true
 #
+# [*step*]
+#   (Optional) The current step in deployment. See tripleo-heat-templates
+#   for more details.
+#   Defaults to hiera('step')
+#
 class tripleo::profile::base::database::mysql (
-  $step                    = hiera('step'),
-  $mysql_server_options    = {},
   $manage_resources        = true,
+  $mysql_server_options    = {},
   $remove_default_accounts = true,
+  $step                    = hiera('step'),
+
 ) {
 
   validate_hash($mysql_server_options)
index 3a5200c..14f6af4 100644 (file)
 #
 # === Parameters
 #
-# [*redis_node_ips*]
-#   (Optional) List of Redis node ips
-#   Defaults to hiera('redis_node_ips')
-#
 # [*bootstrap_nodeid*]
 #   (Optional) Hostname of Redis master
 #   Defaults to hiera('bootstrap_nodeid')
 #
+# [*redis_node_ips*]
+#   (Optional) List of Redis node ips
+#   Defaults to hiera('redis_node_ips')
+#
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
 #
-
 class tripleo::profile::base::database::redis (
-  $redis_node_ips   = hiera('redis_node_ips'),
   $bootstrap_nodeid = hiera('bootstrap_nodeid'),
+  $redis_node_ips   = hiera('redis_node_ips'),
   $step             = hiera('step'),
 ) {
-
   if $step >= 2 {
     if $bootstrap_nodeid == $::hostname {
       $slaveof = undef
index fd43732..845fd41 100644 (file)
 #
 # === Parameters
 #
+# [*glance_backend*]
+#   (Optional) Glance backend(s) to use.
+#   Defaults to downcase(hiera('glance_backend', 'swift'))
+#
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
 #
-# [*glance_backend*]
-#   (Optional) Glance backend(s) to use.
-#   Defaults to downcase(hiera('glance_backend', 'swift'))
-#
 class tripleo::profile::base::glance::api (
-  $step           = hiera('step'),
   $glance_backend = downcase(hiera('glance_backend', 'swift')),
+  $step           = hiera('step'),
 ) {
 
   if $step >= 4 {
index ce89d12..ac6796a 100644 (file)
 #   (Optional) The hostname of the node responsible for bootstrapping tasks
 #   Defaults to hiera('bootstrap_nodeid')
 #
+# [*glance_backend*]
+#   (Optional) Glance backend(s) to use.
+#   Defaults to downcase(hiera('glance_backend', 'swift'))
+#
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
 #
-# [*glance_backend*]
-#   (Optional) Glance backend(s) to use.
-#   Defaults to downcase(hiera('glance_backend', 'swift'))
-
 class tripleo::profile::base::glance::registry (
   $bootstrap_node = hiera('bootstrap_nodeid', undef),
-  $step           = hiera('step'),
   $glance_backend = downcase(hiera('glance_backend', 'swift')),
+  $step           = hiera('step'),
 ) {
   if $::hostname == downcase($bootstrap_node) {
     $sync_db = true
index ee903cb..6a470ca 100644 (file)
 #   for more details.
 #   Defaults to hiera('step')
 #
-
 class tripleo::profile::base::gnocchi (
   $step = hiera('step'),
 ) {
-
   if $step >= 3 {
     include ::gnocchi
     include ::gnocchi::config
index 870a9bb..5e7e215 100644 (file)
 #
 # === Parameters
 #
-# [*gnocchi_backend*]
-#   (Optional) Gnocchi backend string file, swift or rbd
-#   Defaults to swift
-#
 # [*bootstrap_node*]
 #   (Optional) The hostname of the node responsible for bootstrapping tasks
 #   Defaults to hiera('bootstrap_nodeid')
 #
+# [*gnocchi_backend*]
+#   (Optional) Gnocchi backend string file, swift or rbd
+#   Defaults to swift
+#
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
-
+#
 class tripleo::profile::base::gnocchi::api (
-  $gnocchi_backend = downcase(hiera('gnocchi_backend', 'swift')),
   $bootstrap_node  = hiera('bootstrap_nodeid', undef),
+  $gnocchi_backend = downcase(hiera('gnocchi_backend', 'swift')),
   $step            = hiera('step'),
 ) {
   if $::hostname == downcase($bootstrap_node) {
index 29cf882..4d7eb89 100644 (file)
 #   for more details.
 #   Defaults to hiera('step')
 #
-
 class tripleo::profile::base::gnocchi::metricd (
   $step = hiera('step'),
 ) {
-
   include ::tripleo::profile::base::gnocchi
 
   if $step >= 4 {
index 830a128..775b043 100644 (file)
 #   for more details.
 #   Defaults to hiera('step')
 #
-
 class tripleo::profile::base::gnocchi::statsd (
   $step = hiera('step'),
 ) {
-
   include ::tripleo::profile::base::gnocchi
 
   if $step >= 4 {
index 7951941..4f2f850 100644 (file)
 #
 # === Parameters
 #
-# [*step*]
-#   (Optional) The current step in deployment. See tripleo-heat-templates
-#   for more details.
-#   Defaults to hiera('step')
-#
-# [*enable_load_balancer*]
-#   (Optional) Whether or not loadbalancer is enabled.
-#   Defaults to hiera('enable_load_balancer', true).
-#
-# [*generate_service_certificates*]
-#   (Optional) Whether or not certmonger will generate certificates for
-#   HAProxy. This could be as many as specified by the $certificates_specs
-#   variable.
-#   Note that this doesn't configure the certificates in haproxy, it merely
-#   creates the certificates.
-#   Defaults to hiera('generate_service_certificate', false).
-#
-# [*certmonger_ca*]
-#   (Optional) The CA that certmonger will use to generate the certificates.
-#   Defaults to hiera('certmonger_ca', 'local').
-#
 # [*certificates_specs*]
 #   (Optional) The specifications to give to certmonger for the certificate(s)
 #   it will create.
 #         principal: "haproxy/<undercloud fqdn>"
 #   Defaults to {}.
 #
+# [*certmonger_ca*]
+#   (Optional) The CA that certmonger will use to generate the certificates.
+#   Defaults to hiera('certmonger_ca', 'local').
+#
+# [*enable_load_balancer*]
+#   (Optional) Whether or not loadbalancer is enabled.
+#   Defaults to hiera('enable_load_balancer', true).
+#
+# [*generate_service_certificates*]
+#   (Optional) Whether or not certmonger will generate certificates for
+#   HAProxy. This could be as many as specified by the $certificates_specs
+#   variable.
+#   Note that this doesn't configure the certificates in haproxy, it merely
+#   creates the certificates.
+#   Defaults to hiera('generate_service_certificate', false).
+#
+# [*step*]
+#   (Optional) The current step in deployment. See tripleo-heat-templates
+#   for more details.
+#   Defaults to hiera('step')
+#
 class tripleo::profile::base::haproxy (
+  $certificates_specs            = {},
+  $certmonger_ca                 = hiera('certmonger_ca', 'local'),
   $enable_load_balancer          = hiera('enable_load_balancer', true),
-  $step                          = hiera('step'),
   $generate_service_certificates = hiera('generate_service_certificates', false),
-  $certmonger_ca                 = hiera('certmonger_ca', 'local'),
-  $certificates_specs            = {},
+  $step                          = hiera('step'),
 ) {
-
   if $step >= 1 {
     if $enable_load_balancer {
       if str2bool($generate_service_certificates) {
index 1311f20..8512d6e 100644 (file)
 #
 # === Parameters
 #
-# [*step*]
-#   (Optional) The current step in deployment. See tripleo-heat-templates
-#   for more details.
-#   Defaults to hiera('step')
-#
-# [*notification_driver*]
-#   (Optional) Heat notification driver to use.
-#   Defaults to 'messaging'
-#
 # [*bootstrap_master*]
 #   (Optional) The hostname of the node responsible for bootstrapping
 #   Defaults to downcase(hiera('bootstrap_nodeid'))
 #   (Optional) Whether keystone token flushing should be enabled
 #   Defaults to hiera('keystone_enable_db_purge', true)
 #
+# [*notification_driver*]
+#   (Optional) Heat notification driver to use.
+#   Defaults to 'messaging'
+#
+# [*step*]
+#   (Optional) The current step in deployment. See tripleo-heat-templates
+#   for more details.
+#   Defaults to hiera('step')
+#
 class tripleo::profile::base::heat (
-  $step = hiera('step'),
+  $bootstrap_master    = downcase(hiera('bootstrap_nodeid')),
+  $manage_db_purge     = hiera('heat_enable_db_purge', true),
   $notification_driver = 'messaging',
-  $bootstrap_master = downcase(hiera('bootstrap_nodeid')),
-  $manage_db_purge = hiera('heat_enable_db_purge', true),
+  $step                = hiera('step'),
 ) {
-
   # Domain resources will be created at step5 on the pacemaker_master so we
   # configure heat.conf at step3 and 4 but actually create the domain later.
   if $step == 3 or $step == 4 {
index 67a0bfc..68de12b 100644 (file)
@@ -24,9 +24,8 @@
 #   Defaults to hiera('step')
 #
 class tripleo::profile::base::heat::api (
-  $step           = hiera('step'),
+  $step = hiera('step'),
 ) {
-
   if $step >= 4 {
     include ::tripleo::profile::base::heat
     include ::heat::api
index 2813826..7c80fc6 100644 (file)
@@ -24,9 +24,8 @@
 #   Defaults to hiera('step')
 #
 class tripleo::profile::base::heat::api_cfn (
-  $step           = hiera('step'),
+  $step = hiera('step'),
 ) {
-
   if $step >= 4 {
     include ::tripleo::profile::base::heat
     include ::heat::api_cfn
index b2adf92..6362275 100644 (file)
@@ -24,9 +24,8 @@
 #   Defaults to hiera('step')
 #
 class tripleo::profile::base::heat::api_cloudwatch (
-  $step           = hiera('step'),
+  $step = hiera('step'),
 ) {
-
   if $step >= 4 {
     include ::tripleo::profile::base::heat
     include ::heat::api_cloudwatch
index 1fd627f..32a711f 100644 (file)
@@ -26,7 +26,7 @@
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
-
+#
 class tripleo::profile::base::heat::engine (
   $bootstrap_node = hiera('bootstrap_nodeid', undef),
   $step           = hiera('step'),
index ca4f366..f098d37 100644 (file)
@@ -25,7 +25,7 @@
 # [*step*]
 #   (Optional) The current step of the deployment
 #   Defaults to hiera('step')
-
+#
 class tripleo::profile::base::ironic (
   $bootstrap_node = hiera('bootstrap_nodeid', undef),
   $step           = hiera('step'),
index 1406e2d..020cacb 100644 (file)
 #   Defaults to hiera('step')
 #
 class tripleo::profile::base::ironic::api (
-  $step    = hiera('step'),
+  $step = hiera('step'),
 ) {
-
   include ::tripleo::profile::base::ironic
+
   if $step >= 4 {
       include ::ironic::api
   }
index 7a587a5..2cb61fb 100644 (file)
@@ -23,9 +23,8 @@
 #   Defaults to hiera('step')
 #
 class tripleo::profile::base::ironic::conductor (
-  $step    = hiera('step'),
+  $step = hiera('step'),
 ) {
-
   include ::tripleo::profile::base::ironic
 
   if $step >= 4 {
index af7c095..f2063d6 100644 (file)
 #
 # === Parameters
 #
+# [*enable_load_balancer*]
+#   (Optional) Whether or not loadbalancer is enabled.
+#   Defaults to hiera('enable_load_balancer', true).
+#
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
 #
-# [*enable_load_balancer*]
-#   (Optional) Whether or not loadbalancer is enabled.
-#   Defaults to hiera('enable_load_balancer', true).
-#
 class tripleo::profile::base::keepalived (
   $enable_load_balancer = hiera('enable_load_balancer', true),
   $step                 = hiera('step'),
 ) {
-
   if $step >= 1 {
     if $enable_load_balancer and hiera('enable_keepalived', true){
       include ::tripleo::keepalived
     }
   }
-
 }
 
index bba98f8..d8c8e24 100644 (file)
 #
 # === Parameters
 #
-# [*manage_db_purge*]
-#   (Optional) Whether keystone token flushing should be enabled
-#   Defaults to hiera('keystone_enable_db_purge', true)
-#
 # [*bootstrap_node*]
 #   (Optional) The hostname of the node responsible for bootstrapping tasks
 #   Defaults to hiera('bootstrap_nodeid')
 #
+# [*manage_db_purge*]
+#   (Optional) Whether keystone token flushing should be enabled
+#   Defaults to hiera('keystone_enable_db_purge', true)
+#
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
-
+#
 class tripleo::profile::base::keystone (
-  $manage_db_purge = hiera('keystone_enable_db_purge', true),
   $bootstrap_node  = hiera('bootstrap_nodeid', undef),
+  $manage_db_purge = hiera('keystone_enable_db_purge', true),
   $step            = hiera('step'),
 ) {
   if $::hostname == downcase($bootstrap_node) {
index c1188ec..9a3a314 100644 (file)
@@ -24,9 +24,8 @@
 #   Defaults to hiera('step')
 #
 class tripleo::profile::base::manila::api (
-  $step           = hiera('step'),
+  $step = hiera('step'),
 ) {
-
   if $step >= 4 {
     include ::manila
     include ::manila::api
index 3d39c4a..8581187 100644 (file)
@@ -26,7 +26,7 @@
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
-
+#
 class tripleo::profile::base::manila::scheduler (
   $bootstrap_node = hiera('bootstrap_nodeid', undef),
   $step           = hiera('step'),
index 932e013..ed64b29 100644 (file)
 #   Defaults to hiera('step')
 #
 class tripleo::profile::base::manila::share (
-  $step           = hiera('step'),
+  $step = hiera('step'),
 ) {
-
   if $step >= 4 {
     include ::manila::share
   }
-
 }
 
index 54d12c8..72a91e2 100644 (file)
 #   for more details.
 #   Defaults to hiera('step')
 #
-#
 class tripleo::profile::base::memcached (
-  $step             = hiera('step'),
+  $step = hiera('step'),
 ) {
-
   if $step >= 1 {
       include ::memcached
   }
index c0b0538..febbae6 100644 (file)
 #
 # === Parameters
 #
+# [*bootstrap_node*]
+#   (Optional) The hostname of the node responsible for bootstrapping tasks
+#   Defaults to hiera('bootstrap_nodeid')
+#
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
 #
-# [*bootstrap_node*]
-#   (Optional) The hostname of the node responsible for bootstrapping tasks
-#   Defaults to hiera('bootstrap_nodeid')
-#
 class tripleo::profile::base::mistral (
-  $step           = hiera('step'),
   $bootstrap_node = hiera('bootstrap_nodeid', undef),
+  $step           = hiera('step'),
 ) {
-
   if $::hostname == downcase($bootstrap_node) {
     $sync_db = true
   } else {
index 38e1e61..aeff788 100644 (file)
 #
 # === Parameters
 #
+# [*bootstrap_node*]
+#   (Optional) The hostname of the node responsible for bootstrapping tasks
+#   Defaults to hiera('bootstrap_nodeid')
+#
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
 #
-# [*bootstrap_node*]
-#   (Optional) The hostname of the node responsible for bootstrapping tasks
-#   Defaults to hiera('bootstrap_nodeid')
-#
 class tripleo::profile::base::mistral::api (
-  $step           = hiera('step'),
   $bootstrap_node = hiera('bootstrap_nodeid', undef),
+  $step           = hiera('step'),
 ) {
-
   if $::hostname == downcase($bootstrap_node) {
     $sync_db = true
   } else {
index dfcbc2c..cf82237 100644 (file)
 #
 # === Parameters
 #
+# [*bootstrap_node*]
+#   (Optional) The hostname of the node responsible for bootstrapping tasks
+#   Defaults to hiera('bootstrap_nodeid')
+#
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
 #
-# [*bootstrap_node*]
-#   (Optional) The hostname of the node responsible for bootstrapping tasks
-#   Defaults to hiera('bootstrap_nodeid')
-#
 class tripleo::profile::base::mistral::engine (
-  $step           = hiera('step'),
   $bootstrap_node = hiera('bootstrap_nodeid', undef),
+  $step           = hiera('step'),
 ) {
-
   if $::hostname == downcase($bootstrap_node) {
     $sync_db = true
   } else {
index 6f101c8..dd8c642 100644 (file)
 #
 # === Parameters
 #
+# [*bootstrap_node*]
+#   (Optional) The hostname of the node responsible for bootstrapping tasks
+#   Defaults to hiera('bootstrap_nodeid')
+#
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
 #
-# [*bootstrap_node*]
-#   (Optional) The hostname of the node responsible for bootstrapping tasks
-#   Defaults to hiera('bootstrap_nodeid')
-#
 class tripleo::profile::base::mistral::executor (
-  $step           = hiera('step'),
   $bootstrap_node = hiera('bootstrap_nodeid', undef),
-
+  $step           = hiera('step'),
 ) {
-
   if $::hostname == downcase($bootstrap_node) {
     $sync_db = true
   } else {
@@ -44,5 +42,4 @@ class tripleo::profile::base::mistral::executor (
   if $step >= 4 and $sync_db {
     include ::mistral::executor
   }
-
 }
index 1ea7d39..8160452 100644 (file)
@@ -25,7 +25,6 @@
 class tripleo::profile::base::monitoring::fluentd (
   $step = hiera('step', undef)
 ) {
-
   if $step == undef or $step >= 3 {
     include ::fluentd
 
index 4c5ad41..cfd5016 100644 (file)
 #
 # === Parameters
 #
-# [*vhost*]
-#   (Optional) String. RabbitMQ vhost to be used by Sensu
-#   Defaults to '/sensu'
+# [*password*]
+#   (Optional) String. Password to connect to RabbitMQ server
+#   Defaults to hiera('rabbit_password', undef)
 #
 # [*user*]
 #   (Optional) String. Username to connect to RabbitMQ server
 #   Defaults to hiera('rabbit_username', 'sensu')
 #
-# [*password*]
-#   (Optional) String. Password to connect to RabbitMQ server
-#   Defaults to hiera('rabbit_password', undef)
+# [*vhost*]
+#   (Optional) String. RabbitMQ vhost to be used by Sensu
+#   Defaults to '/sensu'
 #
 class tripleo::profile::base::monitoring::rabbitmq (
-  $vhost    = hiera('monitoring_rabbitmq_vhost', '/sensu'),
-  $user     = hiera('monitoring_rabbitmq_username', 'sensu'),
   $password = hiera('monitoring_rabbitmq_password', undef),
+  $user     = hiera('monitoring_rabbitmq_username', 'sensu'),
+  $vhost    = hiera('monitoring_rabbitmq_vhost', '/sensu'),
 ) {
   rabbitmq_vhost { 'sensu-rabbit-vhost':
     ensure => present,
index af09059..a6872b3 100644 (file)
 class tripleo::profile::base::monitoring::sensu (
   $step = hiera('step', undef),
 ) {
-
   if $step == undef or $step >= 3 {
     include ::sensu
-
     package { 'osops-tools-monitoring-oschecks':
       ensure => 'present'
     }
index 7c61f3e..2674b5f 100644 (file)
@@ -25,7 +25,6 @@
 class tripleo::profile::base::monitoring::uchiwa (
   $step = hiera('step', undef),
 ) {
-
   if $step == undef or $step >= 3 {
     include ::uchiwa
   }
index 52cafa9..f2ce94a 100644 (file)
 #
 # === Parameters
 #
-# [*neutron_api_node_ips*]
-#   (Optional) The IPs of the Neutron API hosts
-#   Defaults to hiera('neutron_api_node_ips')
-#
 # [*midonet_libvirt_qemu_data*]
 #   (Optional) qemu.conf data for midonet.
 #   Defaults to hiera('midonet_libvirt_qemu_data')
 #
+# [*neutron_api_node_ips*]
+#   (Optional) The IPs of the Neutron API hosts
+#   Defaults to hiera('neutron_api_node_ips')
+#
 # [*step*]
 #   (Optional) The current step of the deployment
 #   Defaults to hiera('step')
 #
 class tripleo::profile::base::neutron::agents::midonet (
-  $neutron_api_node_ips       = hiera('neutron_api_node_ips', ''),
-  $midonet_libvirt_qemu_data  = hiera('midonet_libvirt_qemu_data', ''),
-  $step                       = hiera('step'),
+  $midonet_libvirt_qemu_data = hiera('midonet_libvirt_qemu_data', ''),
+  $neutron_api_node_ips      = hiera('neutron_api_node_ips', ''),
+  $step                      = hiera('step'),
 ) {
-
   if $step >= 4 {
-
     # TODO(devvesa) provide non-controller ips for these services
     class { '::tripleo::network::midonet::agent':
       zookeeper_servers => $neutron_api_node_ips,
@@ -50,7 +48,5 @@ class tripleo::profile::base::neutron::agents::midonet (
         content => hiera('midonet_libvirt_qemu_data')
       }
     }
-
   }
-
 }
index c50feb2..80beceb 100644 (file)
 #
 # === Parameters
 #
-# [*nova_os_tenant_name*]
-#   (Optional) Nova tenant name
-#   Defaults to hiera('nova_os_tenant_name')
-#
-# [*nova_os_password*]
-#   (Optional) Nova password
-#   Defaults to hiera('nova_password')
+# [*nova_auth_ip*]
+#   (Optional) Nova auth IP
+#   Defaults to hiera('keystone_public_api_virtual_ip')
 #
 # [*nova_metadata_ip*]
 #   (Optional) Nova metadata node IPs
 #   Defaults to hiera('nova_metadata_node_ips')
 #
-# [*nova_auth_ip*]
-#   (Optional) Nova auth IP
-#   Defaults to hiera('keystone_public_api_virtual_ip')
+# [*nova_os_password*]
+#   (Optional) Nova password
+#   Defaults to hiera('nova_password')
+#
+# [*nova_os_tenant_name*]
+#   (Optional) Nova tenant name
+#   Defaults to hiera('nova_os_tenant_name')
 #
 # [*step*]
 #   (Optional) The current step of the deployment
 #   Defaults to hiera('step')
 #
 class tripleo::profile::base::neutron::agents::nuage (
-  $nova_os_tenant_name       = hiera('nova::api::admin_tenant_name', ''),
-  $nova_os_password          = hiera('nova_password', ''),
-  $nova_metadata_ip          = hiera('nova_metadata_node_ips', ''),
-  $nova_auth_ip              = hiera('keystone_public_api_virtual_ip', ''),
-  $step                      = hiera('step'),
+  $nova_auth_ip        = hiera('keystone_public_api_virtual_ip', ''),
+  $nova_metadata_ip    = hiera('nova_metadata_node_ips', ''),
+  $nova_os_password    = hiera('nova_password', ''),
+  $nova_os_tenant_name = hiera('nova::api::admin_tenant_name', ''),
+  $step                = hiera('step'),
 ) {
-
   if $step >= 4 {
-
     include ::nuage::vrs
 
     class { '::nuage::metadataagent':
@@ -56,7 +54,5 @@ class tripleo::profile::base::neutron::agents::nuage (
       nova_metadata_ip    => $nova_metadata_ip,
       nova_auth_ip        => $nova_auth_ip,
     }
-
   }
-
 }
index a313478..24b1a35 100644 (file)
@@ -24,7 +24,7 @@
 #   Defaults to hiera('step')
 #
 class tripleo::profile::base::neutron::dhcp (
-  $step                    = hiera('step'),
+  $step = hiera('step'),
 ) {
   if $step >= 4 {
     include ::tripleo::profile::base::neutron
index 2b57555..14ffa82 100644 (file)
 #
 # === Parameters
 #
-# [*neutron_ovs_use_veth*]
-#   (Optional) Whether to set ovs_use_veth (for older kernel support)
-#   Defaults to hiera('neutron_ovs_use_veth', false)
-#
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
 #
 class tripleo::profile::base::neutron::l3 (
-  $step           = hiera('step'),
+  $step = hiera('step'),
 ) {
   if $step >= 4 {
     include ::tripleo::profile::base::neutron
index d7b4c99..4cc6748 100644 (file)
@@ -24,7 +24,7 @@
 #   Defaults to hiera('step')
 #
 class tripleo::profile::base::neutron::metadata (
-  $step           = hiera('step'),
+  $step = hiera('step'),
 ) {
   if $step >= 4 {
     include ::tripleo::profile::base::neutron
index 3276b82..c8f3044 100644 (file)
 #
 # === Parameters
 #
-# [*vip*]
-#   (Optional) Public Virtual IP Address for this cloud
-#   Defaults to hiera('public_virtual_ip')
+# [*admin_password*]
+#   (Optional) Admin Password for Midonet API
+#   Defaults to hiera('admin_password')
+#
+# [*bind_address*]
+#   (Optional) The address to bind Cassandra and Midonet API to
+#   Defaults to hiera('neutron::bind_host')
 #
 # [*keystone_admin_token*]
 #   (Optional) The Keystone Admin Token
 #   Defaults to hiera('keystone::admin_token')
 #
-# [*zookeeper_client_ip*]
-#   (Optional) The IP of the Zookeeper Client
-#   Defaults to hiera('neutron::bind_host')
-#
-# [*zookeeper_hostnames*]
-#   (Optional) The IPs of the Zookeeper Servers
-#   Defaults to hiera('controller_node_names')
-#
 # [*neutron_api_node_ips*]
 #   (Optional) The IPs of the Neutron API hosts
 #   Defaults to hiera('neutron_api_node_ips')
 #
-# [*bind_address*]
-#   (Optional) The address to bind Cassandra and Midonet API to
-#   Defaults to hiera('neutron::bind_host')
-#
-# [*admin_password*]
-#   (Optional) Admin Password for Midonet API
-#   Defaults to hiera('admin_password')
-#
-# [*zk_on_controller*]
-#   (Optional) Whether to put zookeeper on the controllers
-#   Defaults to hiera('enable_zookeeper_on_controller')
-#
-# [*neutron_auth_tenant*]
-#   (Optional) Tenant to use for Neutron authentication
-#   Defaults to hiera('neutron::server::auth_tenant')
-#
 # [*neutron_auth_password*]
 #   (Optional) Password to use for Neutron authentication
 #   Defaults to hiera('neutron::server::auth_password')
 #
-# [*step*]
-#   (Optional) The current step in deployment. See tripleo-heat-templates
-#   for more details.
-#   Defaults to hiera('step')
+# [*neutron_auth_tenant*]
+#   (Optional) Tenant to use for Neutron authentication
+#   Defaults to hiera('neutron::server::auth_tenant')
 #
 # [*step*]
 #   (Optional) The current step of the deployment
 #   Defaults to hiera('step')
 #
+# [*vip*]
+#   (Optional) Public Virtual IP Address for this cloud
+#   Defaults to hiera('public_virtual_ip')
+#
+# [*zk_on_controller*]
+#   (Optional) Whether to put zookeeper on the controllers
+#   Defaults to hiera('enable_zookeeper_on_controller')
+#
+# [*zookeeper_client_ip*]
+#   (Optional) The IP of the Zookeeper Client
+#   Defaults to hiera('neutron::bind_host')
+#
+# [*zookeeper_hostnames*]
+#   (Optional) The IPs of the Zookeeper Servers
+#   Defaults to hiera('controller_node_names')
+#
 class tripleo::profile::base::neutron::midonet (
-  $vip                   = hiera('public_virtual_ip'),
+  $admin_password        = hiera('admin_password', ''),
+  $bind_address          = hiera('neutron::bind_host', ''),
   $keystone_admin_token  = hiera('keystone::admin_token', ''),
-  $zookeeper_client_ip   = hiera('neutron::bind_host', ''),
-  $zookeeper_hostnames   = hiera('controller_node_names', ''),
   $neutron_api_node_ips  = hiera('neutron_api_node_ips', ''),
-  $bind_address          = hiera('neutron::bind_host', ''),
-  $admin_password        = hiera('admin_password', ''),
-  $zk_on_controller      = hiera('enable_zookeeper_on_controller', ''),
-  $neutron_auth_tenant   = hiera('neutron::server::auth_tenant', ''),
   $neutron_auth_password = hiera('neutron::server::auth_password', ''),
+  $neutron_auth_tenant   = hiera('neutron::server::auth_tenant', ''),
   $step                  = hiera('step'),
+  $vip                   = hiera('public_virtual_ip'),
+  $zk_on_controller      = hiera('enable_zookeeper_on_controller', ''),
+  $zookeeper_client_ip   = hiera('neutron::bind_host', ''),
+  $zookeeper_hostnames   = hiera('controller_node_names', ''),
 ) {
-
   include ::tripleo::profile::base::neutron
   include ::tripleo::profile::base::neutron::agents::midonet
 
index f801511..a4e0cd3 100644 (file)
@@ -26,7 +26,6 @@
 class tripleo::profile::base::neutron::ovs(
   $step           = hiera('step'),
 ) {
-
   include ::tripleo::profile::base::neutron
 
   if $step >= 4 {
index 0193615..c89bc02 100644 (file)
 #
 # === Parameters
 #
-# [*mechanism_drivers*]
-#   (Optional) The mechanism drivers to use with the Ml2 plugin
-#   Defaults to hiera('neutron::plugins::ml2::mechanism_drivers')
-#
 # [*bootstrap_node*]
 #   (Optional) The hostname of the node responsible for bootstrapping tasks
 #   Defaults to hiera('bootstrap_nodeid')
 #
+# [*mechanism_drivers*]
+#   (Optional) The mechanism drivers to use with the Ml2 plugin
+#   Defaults to hiera('neutron::plugins::ml2::mechanism_drivers')
+#
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
-
+#
 class tripleo::profile::base::neutron::plugins::ml2 (
-  $mechanism_drivers  = hiera('neutron::plugins::ml2::mechanism_drivers'),
-  $bootstrap_node     = hiera('bootstrap_nodeid', undef),
-  $step               = hiera('step'),
+  $bootstrap_node    = hiera('bootstrap_nodeid', undef),
+  $mechanism_drivers = hiera('neutron::plugins::ml2::mechanism_drivers'),
+  $step              = hiera('step'),
 ) {
   if $::hostname == downcase($bootstrap_node) {
     $sync_db = true
index 4ada340..0843ec4 100644 (file)
@@ -24,7 +24,7 @@
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
-
+#
 class tripleo::profile::base::neutron::plugins::nuage (
   $bootstrap_node = hiera('bootstrap_nodeid', undef),
   $step           = hiera('step'),
index 6cd710a..fbf46e7 100644 (file)
@@ -24,7 +24,7 @@
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
-
+#
 class tripleo::profile::base::neutron::plugins::opencontrail (
   $bootstrap_node = hiera('bootstrap_nodeid', undef),
   $step           = hiera('step'),
index efd75ac..bc73d29 100644 (file)
@@ -26,7 +26,7 @@
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
-
+#
 class tripleo::profile::base::neutron::plugins::plumgrid (
   $bootstrap_node = hiera('bootstrap_nodeid', undef),
   $step           = hiera('step'),
index 877184d..07c3be1 100644 (file)
 #   (Optional) The hostname of the node responsible for bootstrapping tasks
 #   Defaults to hiera('bootstrap_nodeid')
 #
-# [*step*]
-#   (Optional) The current step of the deployment
-#   Defaults to hiera('step')
+# [*libvirt_enabled*]
+#   (Optional) Whether or not Libvirt is enabled.
+#   Defaults to false
 #
 # [*manage_migration*]
 #   (Optional) Whether or not manage Nova Live migration
 #   Defaults to false
 #
-# [*libvirt_enabled*]
-#   (Optional) Whether or not Libvirt is enabled.
-#   Defaults to false
-#
 # [*nova_compute_enabled*]
 #   (Optional) Whether or not nova-compute is enabled.
 #   Defaults to false
 #
+# [*step*]
+#   (Optional) The current step of the deployment
+#   Defaults to hiera('step')
+#
 class tripleo::profile::base::nova (
   $bootstrap_node       = hiera('bootstrap_nodeid', undef),
-  $step                 = hiera('step'),
-  $manage_migration     = false,
   $libvirt_enabled      = false,
+  $manage_migration     = false,
   $nova_compute_enabled = false,
+  $step                 = hiera('step'),
 ) {
   if $::hostname == downcase($bootstrap_node) {
     $sync_db = true
index a01e90f..c0213fb 100644 (file)
 class tripleo::profile::base::nova::compute::ironic (
   $step = hiera('step'),
 ) {
-
   if $step >= 4 {
     include ::tripleo::profile::base::nova::compute
     include ::nova::compute::ironic
     include ::nova::network::neutron
   }
-
 }
index 5b6f895..956f8ad 100644 (file)
@@ -26,7 +26,6 @@
 class tripleo::profile::base::nova::compute::libvirt (
   $step = hiera('step'),
 ) {
-
   if $step >= 4 {
     include ::tripleo::profile::base::nova::compute
 
index 04c9d06..fa9f12b 100644 (file)
 class tripleo::profile::base::nova::conductor (
   $step = hiera('step'),
 ) {
-
   include ::tripleo::profile::base::nova
   if $step >= 4 {
     include ::nova::conductor
   }
-
 }
index 442cf84..8ccfb8c 100644 (file)
 class tripleo::profile::base::nova::consoleauth (
   $step = hiera('step'),
 ) {
-
   if $step >= 4 {
     include ::tripleo::profile::base::nova
     include ::nova::consoleauth
   }
-
 }
index 29ef372..889b80d 100644 (file)
@@ -26,7 +26,6 @@
 class tripleo::profile::base::nova::libvirt (
   $step = hiera('step'),
 ) {
-
   if $step >= 4 {
     include ::tripleo::profile::base::nova
     include ::nova::compute::libvirt::services
index 13b4e82..3c9b2c2 100644 (file)
 class tripleo::profile::base::nova::scheduler (
   $step = hiera('step'),
 ) {
-
   if $step >= 4 {
     include ::tripleo::profile::base::nova
     include ::nova::scheduler
     include ::nova::scheduler::filter
   }
-
 }
index aa0cc7b..f654fef 100644 (file)
 class tripleo::profile::base::nova::vncproxy (
   $step = hiera('step'),
 ) {
-
   if $step >= 4 {
     include ::tripleo::profile::base::nova
     include ::nova::vncproxy
   }
-
 }
index de3de3c..2c70eab 100644 (file)
@@ -26,7 +26,6 @@
 class tripleo::profile::base::pacemaker (
   $step = hiera('step'),
 ) {
-
   Pcmk_resource <| |> {
     tries     => 10,
     try_sleep => 3,
index 6e86eab..dc6ea4d 100644 (file)
 #
 # === Parameters
 #
-# [*nodes*]
-#   (Optional) Array of host(s) for RabbitMQ nodes.
-#   Defaults to hiera('rabbit_node_ips', []).
-#
-# [*ipv6*]
-#   (Optional) Whether to deploy RabbitMQ on IPv6 network.
-#   Defaults to str2bool(hiera('rabbit_ipv6', false)).
+# [*config_variables*]
+#   (Optional) RabbitMQ environment.
+#   Defaults to hiera('rabbitmq_config_variables').
 #
 # [*environment*]
 #   (Optional) RabbitMQ environment.
 #   Defaults to hiera('rabbitmq_environment').
 #
+# [*ipv6*]
+#   (Optional) Whether to deploy RabbitMQ on IPv6 network.
+#   Defaults to str2bool(hiera('rabbit_ipv6', false)).
+#
 # [*kernel_variables*]
 #   (Optional) RabbitMQ environment.
 #   Defaults to hiera('rabbitmq_environment').
 #
-# [*config_variables*]
-#   (Optional) RabbitMQ environment.
-#   Defaults to hiera('rabbitmq_config_variables').
+# [*nodes*]
+#   (Optional) Array of host(s) for RabbitMQ nodes.
+#   Defaults to hiera('rabbit_node_ips', []).
 #
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   Defaults to hiera('step')
 #
 class tripleo::profile::base::rabbitmq (
-  $nodes            = hiera('rabbit_node_ips', []),
-  $ipv6             = str2bool(hiera('rabbit_ipv6', false)),
+  $config_variables = hiera('rabbitmq_config_variables'),
   $environment      = hiera('rabbitmq_environment'),
+  $ipv6             = str2bool(hiera('rabbit_ipv6', false)),
   $kernel_variables = hiera('rabbitmq_kernel_variables'),
-  $config_variables = hiera('rabbitmq_config_variables'),
+  $nodes            = hiera('rabbit_node_ips', []),
   $step             = hiera('step'),
 ) {
-
   # IPv6 environment, necessary for RabbitMQ.
   if $ipv6 {
     $rabbit_env = merge($environment, {
index dc23d64..df6c4c9 100644 (file)
@@ -24,7 +24,7 @@
 #   Defaults to hiera('step')
 #
 class tripleo::profile::base::sahara::api (
-  $step                    = hiera('step'),
+  $step = hiera('step'),
 ) {
   if $step >= 4 {
     include ::tripleo::profile::base::sahara
index f792a96..db5ec51 100644 (file)
@@ -26,7 +26,7 @@
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
-
+#
 class tripleo::profile::base::sahara::engine (
   $bootstrap_node = hiera('bootstrap_nodeid', undef),
   $step           = hiera('step'),
index 40f7393..2ed6752 100644 (file)
 #
 # === Parameters
 #
-# [*step*]
-#   (Optional) The current step in deployment. See tripleo-heat-templates
-#   for more details.
-#   Defaults to hiera('step')
+# [*snmpd_password*]
+#   The SNMP password
+#   Defaults to hiera('snmpd_readonly_user_password')
 #
 # [*snmpd_user*]
 #   The SNMP username
 #   Defaults to hiera('snmpd_readonly_user_name')
 #
-# [*snmpd_password*]
-#   The SNMP password
-#   Defaults to hiera('snmpd_readonly_user_password')
+# [*step*]
+#   (Optional) The current step in deployment. See tripleo-heat-templates
+#   for more details.
+#   Defaults to hiera('step')
 #
 class tripleo::profile::base::snmp (
-  $step           = hiera('step'),
-  $snmpd_user     = hiera('snmpd_readonly_user_name'),
   $snmpd_password = hiera('snmpd_readonly_user_password'),
+  $snmpd_user     = hiera('snmpd_readonly_user_name'),
+  $step           = hiera('step'),
 ) {
-
   if $step >= 4 {
     snmp::snmpv3_user { $snmpd_user:
       authtype => 'MD5',
index f61f418..bd4c91c 100644 (file)
 #
 # [*swift_zones*]
 #   (Optional) The number of swift zones.
-
+#
 define tripleo::profile::base::swift::add_devices(
   $swift_zones = '1'
 ){
-
   # NOTE(dprince): Swift zones is not yet properly wired into the Heat
   # templates. See: https://review.openstack.org/#/c/97758/3
   # For now our regex supports the r1z1-192.0.2.6:%PORT%/d1 syntax or the
index 40e61ae..1e763a1 100644 (file)
 #   for more details.
 #   Defaults to hiera('step')
 #
-#
 class tripleo::profile::base::swift::proxy (
   $step = hiera('step'),
 ) {
-
   if $step >= 4 {
     include ::swift::proxy
     include ::swift::proxy::proxy_logging
index d94c6be..e0f67cd 100644 (file)
 #
 # === Parameters
 #
-# [*step*]
+#  [*replicas*]
+#    replicas
+#
+#  [*build_ring*] = true,
+#   (Optional) Whether to build the ring
+#   Defaults to true
+#
+#  [*devices*]
+#   (Optional) The swift devices
+#   Defaults to ''
+#
+#  [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
+#
 #  [*swift_zones*]
 #   (Optional) The swift zones
 #   Defaults to 1
-#  [*devices*]
-#   (Optional) The swift devices
-#   Defaults to ''
-#  [*build_ring*]      = true,
-#   (Optional) Whether to build the ring
-#   Defaults to true
-#  [*replicas*]
-#    replicas
-
+#
 class tripleo::profile::base::swift::ringbuilder (
-  $step = hiera('step'),
-  $swift_zones     = '1',
-  $devices         = '',
-  $build_ring      = true,
   $replicas,
+  $build_ring  = true,
+  $devices     = '',
+  $step        = hiera('step'),
+  $swift_zones = '1',
 ) {
-
   if $step >= 2 {
     # pre-install swift here so we can build rings
     include ::swift
index 371cb0d..5e266a9 100644 (file)
 #
 # === Parameters
 #
+# [*enable_swift_storage*]
+#   (Optional) enable_swift_storage
+#   Deprecated: defaults to true
+#
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
 #
-# [*enable_swift_storage*]
-#   (Optional) enable_swift_storage
-#   Deprecated: defaults to true
-#
 class tripleo::profile::base::swift::storage (
-  $step                 = hiera('step'),
   # Deprecated conditional to support ControllerEnableSwiftStorage parameter
   $enable_swift_storage = true,
+  $step                 = hiera('step'),
 ) {
-
   if $step >= 4 {
     if $enable_swift_storage {
       include ::swift::storage::all
index 06edf7f..7a78171 100644 (file)
@@ -26,7 +26,7 @@
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
-
+#
 class tripleo::profile::base::trove::api (
   $bootstrap_node = hiera('bootstrap_nodeid', undef),
   $step           = hiera('step'),
index 4b0b16e..980b3a4 100644 (file)
@@ -31,7 +31,6 @@ class tripleo::profile::pacemaker::apache (
   $bootstrap_node = hiera('bootstrap_nodeid'),
   $step           = hiera('step'),
 ) {
-
   if $::hostname == downcase($bootstrap_node) {
     $pacemaker_master = true
   } else {
index 23021fc..531b4bc 100644 (file)
@@ -31,7 +31,6 @@ class tripleo::profile::pacemaker::ceilometer (
   $bootstrap_node = hiera('bootstrap_nodeid'),
   $step           = hiera('step'),
 ) {
-
   include ::tripleo::profile::base::ceilometer
 
   $ceilometer_backend = downcase(hiera('ceilometer_backend', 'mongodb'))
index e227614..90266be 100644 (file)
@@ -31,7 +31,6 @@ class tripleo::profile::pacemaker::ceilometer::agent::central (
   $pacemaker_master = hiera('bootstrap_nodeid'),
   $step             = hiera('step'),
 ) {
-
   include ::ceilometer::params
   include ::tripleo::profile::pacemaker::ceilometer
   include ::tripleo::profile::base::ceilometer::agent::central
index 868bb22..e419356 100644 (file)
@@ -31,7 +31,6 @@ class tripleo::profile::pacemaker::ceilometer::agent::notification (
   $pacemaker_master = hiera('bootstrap_nodeid'),
   $step             = hiera('step'),
 ) {
-
   include ::ceilometer::params
   include ::tripleo::profile::pacemaker::ceilometer
   include ::tripleo::profile::base::ceilometer::agent::notification
index 3006be8..0eddaec 100644 (file)
@@ -31,7 +31,6 @@ class tripleo::profile::pacemaker::ceilometer::api (
   $pacemaker_master = hiera('bootstrap_nodeid'),
   $step             = hiera('step'),
 ) {
-
   include ::ceilometer::params
   include ::tripleo::profile::pacemaker::ceilometer
   include ::tripleo::profile::base::ceilometer::api
index 6610a07..d0f7217 100644 (file)
 #   for more details.
 #   Defaults to hiera('step')
 #
-# [*sync_db*]
-#   (Optional) Whether to run db sync
-#   Defaults to undef
-#
 class tripleo::profile::pacemaker::ceilometer::collector (
   $pacemaker_master = hiera('bootstrap_nodeid'),
   $step             = hiera('step'),
-  $sync_db          = true,
 ) {
-
   include ::ceilometer::params
   include ::tripleo::profile::pacemaker::ceilometer
-
   include ::tripleo::profile::base::ceilometer::collector
 
   if $step >= 5 and downcase($::hostname) == $pacemaker_master {
index 6354244..d18942d 100644 (file)
@@ -31,7 +31,6 @@ class tripleo::profile::pacemaker::cinder::api (
   $bootstrap_node = hiera('bootstrap_nodeid'),
   $step           = hiera('step'),
 ) {
-
   Service <| tag == 'cinder-service' |> {
     hasrestart => true,
     restart    => '/bin/true',
index 00b0696..e25ef54 100644 (file)
@@ -31,7 +31,6 @@ class tripleo::profile::pacemaker::cinder::scheduler (
   $bootstrap_node = hiera('bootstrap_nodeid'),
   $step           = hiera('step'),
 ) {
-
   Service <| tag == 'cinder-service' |> {
     hasrestart => true,
     restart    => '/bin/true',
index d909831..5a581eb 100644 (file)
@@ -31,7 +31,6 @@ class tripleo::profile::pacemaker::cinder::volume (
   $bootstrap_node = hiera('bootstrap_nodeid'),
   $step           = hiera('step'),
 ) {
-
   Service <| tag == 'cinder::volume' |> {
     hasrestart => true,
     restart    => '/bin/true',
index b8b0781..359a817 100644 (file)
 #   Defaults to hiera('step')
 #
 class tripleo::profile::pacemaker::core (
-  $bootstrap_node       = hiera('bootstrap_nodeid'),
-  $step                 = hiera('step'),
+  $bootstrap_node = hiera('bootstrap_nodeid'),
+  $step           = hiera('step'),
 ) {
-
   if $::hostname == downcase($bootstrap_node) {
     $pacemaker_master = true
   } else {
index 15c84d7..e4b5fcf 100644 (file)
 #
 class tripleo::profile::pacemaker::database::mongodb (
   $mongodb_replset,
-  $bootstrap_node       = hiera('bootstrap_nodeid'),
-  $step                 = hiera('step'),
+  $bootstrap_node = hiera('bootstrap_nodeid'),
+  $step           = hiera('step'),
 ) {
-
   if $step >= 1 {
     include ::mongodb::globals
     include ::mongodb::client
index cc95092..529aeac 100644 (file)
@@ -26,7 +26,6 @@
 class tripleo::profile::pacemaker::database::mysql (
   $step = hiera('step'),
 ) {
-
   if $::hostname == downcase(hiera('bootstrap_nodeid')) {
     $pacemaker_master = true
   } else {
index 27dcbe9..8a37ce9 100644 (file)
@@ -31,7 +31,6 @@
 #   for more details.
 #   Defaults to hiera('step')
 #
-
 class tripleo::profile::pacemaker::database::redis (
   $bootstrap_node       = hiera('bootstrap_nodeid'),
   $enable_load_balancer = hiera('enable_load_balancer', true),
index 91fddf6..664b91f 100644 (file)
 #   (Optional) The hostname of the node responsible for bootstrapping tasks
 #   Defaults to hiera('bootstrap_nodeid')
 #
-# [*step*]
-#   (Optional) The current step in deployment. See tripleo-heat-templates
-#   for more details.
-#   Defaults to hiera('step')
-#
 # [*glance_backend*]
 #   (Optional) Glance backend(s) to use.
 #   Defaults to downcase(hiera('glance_backend', 'swift'))
 #
-# [*glance_file_pcmk_manage*]
-#   (Optional) Whether or not manage glance_file_pcmk.
-#   Defaults to hiera('glance_file_pcmk_manage', false)
-#
 # [*glance_file_pcmk_device*]
 #   (Optional) Device to mount glance file backend.
 #   Defaults to hiera('glance_file_pcmk_device', '')
 #   (Optional) Filesystem type to mount glance file backend.
 #   Defaults to hiera('glance_file_pcmk_fstype', '')
 #
+# [*glance_file_pcmk_manage*]
+#   (Optional) Whether or not manage glance_file_pcmk.
+#   Defaults to hiera('glance_file_pcmk_manage', false)
+#
 # [*glance_file_pcmk_options*]
 #   (Optional) pcmk options to mount Glance file backend..
 #   Defaults to hiera('glance_file_pcmk_options', '')
 #
+# [*step*]
+#   (Optional) The current step in deployment. See tripleo-heat-templates
+#   for more details.
+#   Defaults to hiera('step')
+#
 class tripleo::profile::pacemaker::glance (
   $bootstrap_node             = hiera('bootstrap_nodeid'),
-  $step                       = hiera('step'),
   $glance_backend             = downcase(hiera('glance_backend', 'swift')),
-  $glance_file_pcmk_manage    = hiera('glance_file_pcmk_manage', false),
   $glance_file_pcmk_device    = hiera('glance_file_pcmk_device', ''),
   $glance_file_pcmk_directory = hiera('glance_file_pcmk_directory', ''),
   $glance_file_pcmk_fstype    = hiera('glance_file_pcmk_fstype', ''),
+  $glance_file_pcmk_manage    = hiera('glance_file_pcmk_manage', false),
   $glance_file_pcmk_options   = hiera('glance_file_pcmk_options', ''),
+  $step                       = hiera('step'),
 ) {
-
   Service <| tag == 'glance-service' |> {
     hasrestart => true,
     restart    => '/bin/true',
index c8630ce..5bfc174 100644 (file)
 #
 # === Parameters
 #
-# [*gnocchi_indexer_backend*]
-#   (Optional) Gnocchi indexer backend
-#   Defaults to mysql
-#
 # [*bootstrap_node*]
 #   (Optional) The hostname of the node responsible for bootstrapping tasks
 #   Defaults to hiera('bootstrap_nodeid')
 #
+# [*gnocchi_indexer_backend*]
+#   (Optional) Gnocchi indexer backend
+#   Defaults to mysql
+#
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
 #
-
 class tripleo::profile::pacemaker::gnocchi (
+  $bootstrap_node          = hiera('bootstrap_nodeid'),
   $gnocchi_indexer_backend = downcase(hiera('gnocchi_indexer_backend', 'mysql')),
-  $bootstrap_node  = hiera('bootstrap_nodeid'),
-  $step            = hiera('step'),
+  $step                    = hiera('step'),
 ) {
-
   Service <| tag == 'gnocchi-service' |> {
     hasrestart => true,
     restart    => '/bin/true',
index ede4c9a..29f2435 100644 (file)
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
-
+#
 class tripleo::profile::pacemaker::gnocchi::api (
-  $step             = hiera('step'),
+  $step = hiera('step'),
 ) {
-
   include ::tripleo::profile::pacemaker::gnocchi
   include ::tripleo::profile::pacemaker::apache
-
-  class { '::tripleo::profile::base::gnocchi::api':
-    step    => $step,
-  }
+  include ::tripleo::profile::base::gnocchi::api
 }
index 4ac5e74..c9dc2d9 100644 (file)
 #   for more details.
 #   Defaults to hiera('step')
 #
-
 class tripleo::profile::pacemaker::gnocchi::metricd (
   $pacemaker_master = hiera('bootstrap_nodeid'),
   $step             = hiera('step'),
 ) {
-
   include ::gnocchi::params
   include ::tripleo::profile::pacemaker::gnocchi
 
index 526dcac..42d30b9 100644 (file)
 #   for more details.
 #   Defaults to hiera('step')
 #
-
 class tripleo::profile::pacemaker::gnocchi::statsd (
   $pacemaker_master = hiera('bootstrap_nodeid'),
   $step             = hiera('step'),
 ) {
-
   include ::gnocchi::params
   include ::tripleo::profile::pacemaker::gnocchi
 
index a7aca58..1b83d9b 100644 (file)
 #   (Optional) The hostname of the node responsible for bootstrapping tasks
 #   Defaults to hiera('bootstrap_nodeid')
 #
+# [*enable_load_balancer*]
+#   (Optional) Whether load balancing is enabled for this cluster
+#   Defaults to hiera('enable_load_balancer', true)
+#
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
 #
-# [*enable_load_balancer*]
-#   (Optional) Whether load balancing is enabled for this cluster
-#   Defaults to hiera('enable_load_balancer', true)
-#
 class tripleo::profile::pacemaker::haproxy (
   $bootstrap_node       = hiera('bootstrap_nodeid'),
+  $enable_load_balancer = hiera('enable_load_balancer', true),
   $step                 = hiera('step'),
-  $enable_load_balancer = hiera('enable_load_balancer', true)
 ) {
-
   include ::tripleo::profile::base::haproxy
 
   if $::hostname == downcase($bootstrap_node) {
index e72f02a..e3c1598 100644 (file)
 #
 # === Parameters
 #
-# [*bootstrap_node*]
-#   (Optional) The hostname of the node responsible for bootstrapping tasks
-#   Defaults to hiera('bootstrap_nodeid')
-#
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
 #
 class tripleo::profile::pacemaker::heat (
-  $bootstrap_node             = hiera('bootstrap_nodeid'),
-  $step                       = hiera('step'),
+  $step = hiera('step'),
 ) {
-
   Service <| tag == 'heat-service' |> {
     hasrestart => true,
     restart    => '/bin/true',
@@ -39,8 +33,6 @@ class tripleo::profile::pacemaker::heat (
     stop       => '/bin/true',
   }
 
-  class { '::tripleo::profile::base::heat':
-    bootstrap_master         => $bootstrap_node,
-  }
+  include ::tripleo::profile::base::heat
 
 }
index ed9ca5b..0fc4f8a 100644 (file)
 #   Defaults to hiera('step')
 #
 class tripleo::profile::pacemaker::heat::api (
-  $bootstrap_node             = hiera('bootstrap_nodeid'),
-  $step                       = hiera('step'),
+  $bootstrap_node = hiera('bootstrap_nodeid'),
+  $step           = hiera('step'),
 ) {
-
   if $::hostname == downcase($bootstrap_node) {
     $pacemaker_master = true
   } else {
index d3ba6e4..1230c6b 100644 (file)
 #   Defaults to hiera('step')
 #
 class tripleo::profile::pacemaker::heat::api_cfn (
-  $bootstrap_node             = hiera('bootstrap_nodeid'),
-  $step                       = hiera('step'),
+  $bootstrap_node = hiera('bootstrap_nodeid'),
+  $step           = hiera('step'),
 ) {
-
   if $::hostname == downcase($bootstrap_node) {
     $pacemaker_master = true
   } else {
index a7784fb..6110a0c 100644 (file)
 #   Defaults to hiera('step')
 #
 class tripleo::profile::pacemaker::heat::api_cloudwatch (
-  $bootstrap_node             = hiera('bootstrap_nodeid'),
-  $step                       = hiera('step'),
+  $bootstrap_node = hiera('bootstrap_nodeid'),
+  $step           = hiera('step'),
 ) {
-
   if $::hostname == downcase($bootstrap_node) {
     $pacemaker_master = true
   } else {
index 158c138..88744ad 100644 (file)
 #   Defaults to hiera('step')
 #
 class tripleo::profile::pacemaker::heat::engine (
-  $bootstrap_node             = hiera('bootstrap_nodeid'),
-  $step                       = hiera('step'),
+  $bootstrap_node = hiera('bootstrap_nodeid'),
+  $step           = hiera('step'),
 ) {
-
   if $::hostname == downcase($bootstrap_node) {
     $pacemaker_master = true
   } else {
index f48193a..db14aea 100644 (file)
 #   (Optional) The hostname of the node responsible for bootstrapping tasks
 #   Defaults to hiera('bootstrap_nodeid')
 #
+# [*enable_load_balancer*]
+#   (Optional) Whether load balancing is enabled for this cluster
+#   Defaults to hiera('enable_load_balancer', true)
+#
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
 #
-# [*enable_load_balancer*]
-#   (Optional) Whether load balancing is enabled for this cluster
-#   Defaults to hiera('enable_load_balancer', true)
-#
 class tripleo::profile::pacemaker::keystone (
   $bootstrap_node       = hiera('bootstrap_nodeid'),
+  $enable_load_balancer = hiera('enable_load_balancer', true),
   $step                 = hiera('step'),
-  $enable_load_balancer = hiera('enable_load_balancer', true)
 ) {
-
   Service <| tag == 'keystone-service' |> {
     hasrestart => true,
     restart    => '/bin/true',
index f3666c2..43ae875 100644 (file)
 #   (Optional) The hostname of the node responsible for bootstrapping tasks
 #   Defaults to hiera('bootstrap_nodeid')
 #
-# [*step*]
-#   (Optional) The current step in deployment. See tripleo-heat-templates
-#   for more details.
-#   Defaults to hiera('step')
+# [*cinder_volume_type*]
+#   (Optional)
+#   Defaults to hiera('manila::backend::generic::cinder_volume_type', '')
+#
+# [*driver_handles_share_servers*]
+#   (Optional)
+#   Defaults to hiera('manila::backend::generic::driver_handles_share_servers')
 #
 # [*manila_generic_enable*]
 #   (Optional) Enable the generic backend.
 #   Defaults to hiera('manila_generic_enable_backend', 'false')
 #
-# [*driver_handles_share_servers*]
+# [*max_time_to_attach*]
 #   (Optional)
-#   Defaults to hiera('manila::backend::generic::driver_handles_share_servers')
+#   Defaults to hiera('manila::backend::generic::max_time_to_attach')
 #
-# [*smb_template_config_path*]
+# [*max_time_to_create_volume*]
 #   (Optional)
-#   Defaults to hiera('manila::backend::generic::smb_template_config_path')
+#   Defaults to hiera('manila::backend::generic::max_time_to_create_volume')
 #
-# [*volume_name_template*]
-#   (Optional)
-#   Defaults to hiera('manila::backend::generic::volume_name_template')
-
-# [*volume_snapshot_name_template*]
+# [*service_instance_flavor_id*]
 #   (Optional)
-#   Defaults to hiera('manila::backend::generic::volume_snapshot_name_template')
+#   Defaults to hiera('manila::service_instance::service_instance_flavor_id')
 #
-# [*share_mount_path*]
+# [*service_instance_password*]
 #   (Optional)
-#   Defaults to hiera('manila::backend::generic::share_mount_path')
+#   Defaults to hiera('manila::service_instance::service_instance_password')
 #
-# [*max_time_to_create_volume*]
+# [*service_instance_smb_config_path*]
 #   (Optional)
-#   Defaults to hiera('manila::backend::generic::max_time_to_create_volume')
+#   Defaults to downcase(hiera('manila::backend::generic::service_instance_smb_config_path'))
 #
-# [*max_time_to_attach*]
+# [*service_instance_user*]
 #   (Optional)
-#   Defaults to hiera('manila::backend::generic::max_time_to_attach')
+#   Defaults to hiera('manila::service_instance::service_instance_user')
 #
-# [*service_instance_smb_config_path*]
+# [*share_mount_path*]
 #   (Optional)
-#   Defaults to downcase(hiera('manila::backend::generic::service_instance_smb_config_path'))
+#   Defaults to hiera('manila::backend::generic::share_mount_path')
 #
 # [*share_volume_fstype*]
 #   (Optional)
 #   Defaults to hiera('manila::backend::generic::share_volume_fstype')
 #
-# [*cinder_volume_type*]
+# [*smb_template_config_path*]
 #   (Optional)
-#   Defaults to hiera('manila::backend::generic::cinder_volume_type', '')
+#   Defaults to hiera('manila::backend::generic::smb_template_config_path')
 #
-# [*service_instance_user*]
-#   (Optional)
-#   Defaults to hiera('manila::service_instance::service_instance_user')
-
-# [*service_instance_password*]
+# [*step*]
+#   (Optional) The current step in deployment. See tripleo-heat-templates
+#   for more details.
+#   Defaults to hiera('step')
+#
+# [*volume_name_template*]
 #   (Optional)
-#   Defaults to hiera('manila::service_instance::service_instance_password')
-
-# [*service_instance_flavor_id*]
+#   Defaults to hiera('manila::backend::generic::volume_name_template')
+#
+# [*volume_snapshot_name_template*]
 #   (Optional)
-#   Defaults to hiera('manila::service_instance::service_instance_flavor_id')
+#   Defaults to hiera('manila::backend::generic::volume_snapshot_name_template')
 #
 class tripleo::profile::pacemaker::manila (
   $bootstrap_node                   = hiera('bootstrap_nodeid'),
-  $step                             = hiera('step'),
-  $manila_generic_enable            = hiera('manila_generic_enable_backend', false),
+  $cinder_volume_type               = hiera('manila::backend::generic::cinder_volume_type', ''),
   $driver_handles_share_servers     = hiera('manila::backend::generic::driver_handles_share_servers'),
-  $smb_template_config_path         = hiera('manila::backend::generic::smb_template_config_path'),
-  $volume_name_template             = hiera('manila::backend::generic::volume_name_template'),
-  $volume_snapshot_name_template    = hiera('manila::backend::generic::volume_snapshot_name_template'),
-  $share_mount_path                 = hiera('manila::backend::generic::share_mount_path'),
-  $max_time_to_create_volume        = hiera('manila::backend::generic::max_time_to_create_volume'),
+  $manila_generic_enable            = hiera('manila_generic_enable_backend', false),
   $max_time_to_attach               = hiera('manila::backend::generic::max_time_to_attach'),
+  $max_time_to_create_volume        = hiera('manila::backend::generic::max_time_to_create_volume'),
+  $service_instance_flavor_id       = hiera('manila::service_instance::service_instance_flavor_id'),
+  $service_instance_password        = hiera('manila::service_instance::service_instance_password'),
   $service_instance_smb_config_path = hiera('manila::backend::generic::service_instance_smb_config_path'),
-  $share_volume_fstype              = hiera('manila::backend::generic::share_volume_fstype'),
-  $cinder_volume_type               = hiera('manila::backend::generic::cinder_volume_type', ''),
   $service_instance_user            = hiera('manila::service_instance::service_instance_user'),
-  $service_instance_password        = hiera('manila::service_instance::service_instance_password'),
-  $service_instance_flavor_id       = hiera('manila::service_instance::service_instance_flavor_id'),
+  $share_mount_path                 = hiera('manila::backend::generic::share_mount_path'),
+  $share_volume_fstype              = hiera('manila::backend::generic::share_volume_fstype'),
+  $smb_template_config_path         = hiera('manila::backend::generic::smb_template_config_path'),
+  $step                             = hiera('step'),
+  $volume_name_template             = hiera('manila::backend::generic::volume_name_template'),
+  $volume_snapshot_name_template    = hiera('manila::backend::generic::volume_snapshot_name_template'),
 ) {
-
   if $::hostname == downcase($bootstrap_node) {
     $pacemaker_master = true
   } else {
index 09af5d6..2a6bd4d 100644 (file)
@@ -31,7 +31,6 @@ class tripleo::profile::pacemaker::memcached (
   $bootstrap_node = hiera('bootstrap_nodeid'),
   $step           = hiera('step'),
 ) {
-
   if $::hostname == downcase($bootstrap_node) {
     $pacemaker_master = true
   } else {
index 0298298..6525126 100644 (file)
 #
 # === Parameters
 #
-# [*step*]
-#   (Optional) The step in the deployment
-#   Defaults to hiera('step')
-#
-# [*pacemaker_master*]
-#   (Optional) The hostname of the pacemaker master
-#   Defaults to hiera('bootstrap_nodeid', undef)
+# [*enable_dhcp*]
+#   (Optional) Whether to include the Neutron DHCP agent pacemaker profile
+#   Defaults to hiera('neutron::enable_dhcp_agent', false)
 #
 # [*enable_l3*]
 #   (Optional) Whether to include the Neutron L3 agent pacemaker profile
 #   Defaults to hiera('neutron::enable_l3_agent', false)
 #
-# [*enable_dhcp*]
-#   (Optional) Whether to include the Neutron DHCP agent pacemaker profile
-#   Defaults to hiera('neutron::enable_dhcp_agent', false)
-#
 # [*enable_metadata*]
 #   (Optional) Whether to include the Neutron Metadata agent pacemaker profile
 #   Defaults to hiera('neutron::enable_metadata_agent', false)
 #   (Optional) Whether to include the Neutron OVS agent pacemaker profile
 #   Defaults to hiera('neutron::enable_ovs_agent', false)
 #
+# [*pacemaker_master*]
+#   (Optional) The hostname of the pacemaker master
+#   Defaults to hiera('bootstrap_nodeid', undef)
+#
+# [*step*]
+#   (Optional) The step in the deployment
+#   Defaults to hiera('step')
+#
 class tripleo::profile::pacemaker::neutron (
-  $step             = hiera('step'),
-  $pacemaker_master = hiera('bootstrap_nodeid', undef),
   # We can drop the hiera defaults once the neutron roles are decomposed
-  $enable_l3        = hiera('neutron::enable_l3_agent', false),
   $enable_dhcp      = hiera('neutron::enable_dhcp_agent', false),
+  $enable_l3        = hiera('neutron::enable_l3_agent', false),
   $enable_metadata  = hiera('neutron::enable_metadata_agent', false),
   $enable_ovs       = hiera('neutron::enable_ovs_agent', false),
+  #Don't drop below this line
+  $pacemaker_master = hiera('bootstrap_nodeid', undef),
+  $step             = hiera('step'),
 ) {
-
   Service <|
     tag == 'neutron-service'
   |> {
index d738f21..e76012f 100644 (file)
 #
 # === Parameters
 #
+# [*pacemaker_master*]
+#   (Optional) The hostname of the pacemaker master
+#   Defaults to hiera('bootstrap_nodeid', undef)
+#
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
 #
-# [*pacemaker_master*]
-#   (Optional) The hostname of the pacemaker master
-#   Defaults to hiera('bootstrap_nodeid', undef)
-#
 class tripleo::profile::pacemaker::neutron::dhcp (
-  $step             = hiera('step'),
   $pacemaker_master = hiera('bootstrap_nodeid', undef),
+  $step             = hiera('step'),
 ) {
-
   include ::neutron::params
   include ::tripleo::profile::pacemaker::neutron
   include ::tripleo::profile::base::neutron::dhcp
index fec4af2..c3ae3b8 100644 (file)
 #
 # === Parameters
 #
+# [*pacemaker_master*]
+#   (Optional) The hostname of the pacemaker master
+#   Defaults to hiera('bootstrap_nodeid', undef)
+#
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
 #
-# [*pacemaker_master*]
-#   (Optional) The hostname of the pacemaker master
-#   Defaults to hiera('bootstrap_nodeid', undef)
-#
 class tripleo::profile::pacemaker::neutron::l3 (
-  $step             = hiera('step'),
   $pacemaker_master = hiera('bootstrap_nodeid', undef),
+  $step             = hiera('step'),
 ) {
-
   include ::neutron::params
   include ::tripleo::profile::pacemaker::neutron
   include ::tripleo::profile::base::neutron::l3
index 4fccea2..f09edba 100644 (file)
 #
 # === Parameters
 #
+# [*pacemaker_master*]
+#   (Optional) The hostname of the pacemaker master
+#   Defaults to hiera('bootstrap_nodeid', undef)
+#
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
 #
-# [*pacemaker_master*]
-#   (Optional) The hostname of the pacemaker master
-#   Defaults to hiera('bootstrap_nodeid', undef)
-#
 class tripleo::profile::pacemaker::neutron::metadata (
-  $step             = hiera('step'),
   $pacemaker_master = hiera('bootstrap_nodeid', undef),
+  $step             = hiera('step'),
 ) {
-
   include ::neutron::params
   include ::tripleo::profile::pacemaker::neutron
   include ::tripleo::profile::base::neutron::metadata
index 9d380f0..453641a 100644 (file)
 #
 # === Parameters
 #
+# [*pacemaker_master*]
+#   (Optional) The hostname of the pacemaker master
+#   Defaults to hiera('bootstrap_nodeid', undef)
+#
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
 #
-# [*pacemaker_master*]
-#   (Optional) The hostname of the pacemaker master
-#   Defaults to hiera('bootstrap_nodeid', undef)
-#
 class tripleo::profile::pacemaker::neutron::midonet (
-  $step             = hiera('step'),
   $pacemaker_master = hiera('bootstrap_nodeid', undef),
+  $step             = hiera('step'),
 ) {
   include ::neutron::params
   include ::tripleo::profile::pacemaker::neutron
index a330de1..7e3b15c 100644 (file)
 #
 # === Parameters
 #
+# [*pacemaker_master*]
+#   (Optional) The hostname of the pacemaker master
+#   Defaults to hiera('bootstrap_nodeid', undef)
+#
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
 #
-# [*pacemaker_master*]
-#   (Optional) The hostname of the pacemaker master
-#   Defaults to hiera('bootstrap_nodeid', undef)
-#
 class tripleo::profile::pacemaker::neutron::ovs (
-  $step             = hiera('step'),
   $pacemaker_master = hiera('bootstrap_nodeid', undef),
+  $step             = hiera('step'),
 ) {
-
   include ::neutron::params
   include ::tripleo::profile::pacemaker::neutron
   include ::tripleo::profile::base::neutron::ovs
index e5e13b7..aff682a 100644 (file)
 #
 class tripleo::profile::pacemaker::neutron::plugins::ml2
 {
-
   include ::neutron::params
   include ::tripleo::profile::pacemaker::neutron
-
   include ::tripleo::profile::base::neutron::plugins::ml2
-
 }
index de00761..03cdb7e 100644 (file)
@@ -20,7 +20,5 @@
 #
 class tripleo::profile::pacemaker::neutron::plugins::nuage
 {
-
   include ::tripleo::profile::base::neutron::plugins::nuage
-
 }
index f1568e9..438245a 100644 (file)
@@ -20,7 +20,5 @@
 #
 class tripleo::profile::pacemaker::neutron::plugins::opencontrail
 {
-
   include ::tripleo::profile::base::neutron::plugins::opencontrail
-
 }
index 7313ef9..38b2179 100644 (file)
@@ -20,7 +20,5 @@
 #
 class tripleo::profile::pacemaker::neutron::plugins::plumgrid
 {
-
   include ::tripleo::profile::base::neutron::plugins::plumgrid
-
 }
index acd7993..d817ee7 100644 (file)
@@ -31,7 +31,6 @@ class tripleo::profile::pacemaker::neutron::server (
   $pacemaker_master = hiera('bootstrap_nodeid', undef),
   $step             = hiera('step'),
 ) {
-
   include ::neutron::params
   include ::tripleo::profile::pacemaker::neutron
 
index 62a8042..222035e 100644 (file)
@@ -25,7 +25,6 @@
 class tripleo::profile::pacemaker::nova (
   $step = hiera('step'),
 ) {
-
   Service <|
     tag == 'nova-service'
   |> {
index 5e8f15f..188beda 100644 (file)
 #
 # === Parameters
 #
+# [*pacemaker_master*]
+#   (Optional) The hostname of the pacemaker master
+#   Defaults to downcase(hiera('bootstrap_nodeid'))
+#
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
 #
-# [*pacemaker_master*]
-#   (Optional) The hostname of the pacemaker master
-#   Defaults to downcase(hiera('bootstrap_nodeid'))
-#
 class tripleo::profile::pacemaker::nova::api (
-  $step             = hiera('step'),
   $pacemaker_master = downcase(hiera('bootstrap_nodeid')),
+  $step             = hiera('step'),
 ) {
 
   include ::nova::params
index 3e390e0..f2605cb 100644 (file)
 #
 # === Parameters
 #
+# [*pacemaker_master*]
+#   (Optional) The hostname of the pacemaker master
+#   Defaults to hiera('bootstrap_nodeid')
+#
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
 #
-# [*pacemaker_master*]
-#   (Optional) The hostname of the pacemaker master
-#   Defaults to hiera('bootstrap_nodeid')
-#
 class tripleo::profile::pacemaker::nova::conductor (
-  $step             = hiera('step'),
   $pacemaker_master = hiera('bootstrap_nodeid'),
+  $step             = hiera('step'),
 ) {
 
   include ::nova::params
index ad538c4..6cd8c15 100644 (file)
 #
 # === Parameters
 #
+# [*pacemaker_master*]
+#   (Optional) The hostname of the pacemaker master
+#   Defaults to hiera('bootstrap_nodeid')
+#
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
 #
-# [*pacemaker_master*]
-#   (Optional) The hostname of the pacemaker master
-#   Defaults to hiera('bootstrap_nodeid')
-#
 class tripleo::profile::pacemaker::nova::consoleauth (
-  $step             = hiera('step'),
   $pacemaker_master = hiera('bootstrap_nodeid'),
+  $step             = hiera('step'),
 ) {
 
   include ::nova::params
index 9e78c30..8c387d2 100644 (file)
 #
 # === Parameters
 #
+# [*pacemaker_master*]
+#   (Optional) The hostname of the pacemaker master
+#   Defaults to hiera('bootstrap_nodeid')
+#
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
 #
-# [*pacemaker_master*]
-#   (Optional) The hostname of the pacemaker master
-#   Defaults to hiera('bootstrap_nodeid')
-#
 class tripleo::profile::pacemaker::nova::scheduler (
-  $step             = hiera('step'),
   $pacemaker_master = hiera('bootstrap_nodeid'),
+  $step             = hiera('step'),
 ) {
-
   include ::nova::params
   include ::tripleo::profile::pacemaker::nova
   include ::tripleo::profile::base::nova::scheduler
index ec71ca7..3652daa 100644 (file)
 #
 # === Parameters
 #
+# [*pacemaker_master*]
+#   (Optional) The hostname of the pacemaker master
+#   Defaults to hiera('bootstrap_nodeid')
+#
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
 #
-# [*pacemaker_master*]
-#   (Optional) The hostname of the pacemaker master
-#   Defaults to hiera('bootstrap_nodeid')
-#
 class tripleo::profile::pacemaker::nova::vncproxy (
-  $step             = hiera('step'),
   $pacemaker_master = hiera('bootstrap_nodeid'),
+  $step             = hiera('step'),
 ) {
-
   include ::nova::params
   include ::tripleo::profile::pacemaker::nova
   include ::tripleo::profile::base::nova::vncproxy
index efb91b5..93edec9 100644 (file)
 #   (Optional) The hostname of the node responsible for bootstrapping tasks
 #   Defaults to hiera('bootstrap_nodeid')
 #
+# [*erlang_cookie*]
+#   (Optional) Content of erlang cookie.
+#   Defaults to hiera('rabbitmq::erlang_cookie').
+#
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
 #   for more details.
 #   Defaults to hiera('step')
 #
-# [*erlang_cookie*]
-#   (Optional) Content of erlang cookie.
-#   Defaults to hiera('rabbitmq::erlang_cookie').
-#
 class tripleo::profile::pacemaker::rabbitmq (
   $bootstrap_node = hiera('bootstrap_nodeid'),
-  $step           = hiera('step'),
   $erlang_cookie  = hiera('rabbitmq::erlang_cookie'),
+  $step           = hiera('step'),
 ) {
-
   if $::hostname == downcase($bootstrap_node) {
     $pacemaker_master = true
   } else {
index 0627017..07cd882 100644 (file)
@@ -26,7 +26,6 @@
 class tripleo::profile::pacemaker::sahara (
   $step = hiera('step'),
 ) {
-
   Service <| tag == 'sahara-service' |> {
     hasrestart => true,
     restart    => '/bin/true',
index eed82dd..0e3d97a 100644 (file)
 #   Defaults to hiera('step')
 #
 class tripleo::profile::pacemaker::sahara::api (
-  $bootstrap_node       = hiera('bootstrap_nodeid'),
-  $step                 = hiera('step'),
+  $bootstrap_node = hiera('bootstrap_nodeid'),
+  $step           = hiera('step'),
 ) {
-
   if $::hostname == downcase($bootstrap_node) {
     $pacemaker_master = true
   } else {
index 850c645..ada6c06 100644 (file)
 #   Defaults to hiera('step')
 #
 class tripleo::profile::pacemaker::sahara::engine (
-  $bootstrap_node       = hiera('bootstrap_nodeid'),
-  $step                 = hiera('step'),
+  $bootstrap_node = hiera('bootstrap_nodeid'),
+  $step           = hiera('step'),
 ) {
-
   if $::hostname == downcase($bootstrap_node) {
     $pacemaker_master = true
   } else {