Merge "Replace hiera('service_names') by hiera('enabled_services')"

manifests/firewall.pp

@@ -151,7 +151,7 @@ class tripleo::firewall(
     #     dport: 999
     #     proto: udp
     #     action: accept
-    $service_names = hiera('service_names', [])
+    $service_names = hiera('enabled_services', [])
     tripleo::firewall::service_rules { $service_names: }
 
 

manifests/profile/base/logging/fluentd.pp

@@ -116,7 +116,7 @@ class tripleo::profile::base::logging::fluentd (
   $fluentd_monitoring = true,
   $fluentd_monitoring_bind = '127.0.0.1',
   $fluentd_monitoring_port = 24220,
-  $service_names = hiera('service_names', []),
+  $service_names = hiera('enabled_services', []),
   $fluentd_managed_rsyslog = false
 ) {
   if $step >= 4 {

manifests/profile/base/logging/rsyslog.pp

@@ -25,7 +25,7 @@
 # [*service_names*]
 #   (Optional) List of services enabled on the current role. This is used
 #   to obtain per-service configuration information.
-#   Defaults to hiera('service_names', [])
+#   Defaults to hiera('enabled_services', [])
 #
 # [*elasticsearch*]
 #   (Optional) Hash. Configuration for output plugin omelasticsearch.
@@ -47,7 +47,7 @@
 #   Defaults to undef
 class tripleo::profile::base::logging::rsyslog (
   $step                          = Integer(hiera('step')),
-  $service_names                 = hiera('service_names', []),
+  $service_names                 = hiera('enabled_services', []),
   $elasticsearch                 = undef,
   $elasticsearch_tls_ca_cert     = undef,
   $elasticsearch_tls_client_cert = undef,

manifests/profile/base/metrics/collectd.pp

@@ -231,7 +231,7 @@ class tripleo::profile::base::metrics::collectd (
   $amqp_instances = {},
   $amqp_retry_delay = undef,
   $amqp_interval = undef,
-  $service_names = hiera('service_names', []),
+  $service_names = hiera('enabled_services', []),
   $collectd_manage_repo = false,
   $python_read_plugins = []
 ) {

manifests/profile/base/neutron/plugins/ml2.pp

@@ -31,7 +31,7 @@
 #   We may not want to configure a ml2 plugin for a role,
 #   in spite of the fact that it is in the the drivers list.
 #   Check if the required service is enabled from the service list.
-#   Defaults to hiera('service_names')
+#   Defaults to hiera('enabled_services')
 #
 # [*step*]
 #   (Optional) The current step in deployment. See tripleo-heat-templates
@@ -41,7 +41,7 @@
 class tripleo::profile::base::neutron::plugins::ml2 (
   $bootstrap_node    = hiera('neutron_plugin_ml2_short_bootstrap_node_name', undef),
   $mechanism_drivers = hiera('neutron::plugins::ml2::mechanism_drivers'),
-  $service_names     = hiera('service_names'),
+  $service_names     = hiera('enabled_services'),
   $step              = Integer(hiera('step')),
 ) {
   if $::hostname == downcase($bootstrap_node) {

manifests/profile/base/nova/migration/target.pp

@@ -36,7 +36,7 @@
 #   (Optional) List of services enabled on the current role.
 #   If the nova_migration_target service is not enabled then migration over
 #   ssh will be disabled.
-#   Defaults to hiera('service_names', [])
+#   Defaults to hiera('enabled_services', [])
 #
 # [*wrapper_command*]
 #   (Internal) Used to override the wrapper command when proxying
@@ -46,7 +46,7 @@ class tripleo::profile::base::nova::migration::target (
   $step                = Integer(hiera('step')),
   $ssh_authorized_keys = [],
   $ssh_localaddrs      = [],
-  $services_enabled    = hiera('service_names', []),
+  $services_enabled    = hiera('enabled_services', []),
   $wrapper_command     = '/bin/nova-migration-wrapper',
 ) {
 

spec/fixtures/hieradata/step2.yaml

@@ -1,7 +1,7 @@
 ---
 step: 2
 # rsyslog tests
-service_names:
+enabled_services:
  - 'horizon'
 tripleo_logging_sources_horizon:
  - file: '/var/log/horizon/test.log'

spec/fixtures/hieradata/step4.yaml

@@ -11,7 +11,7 @@ cinder::backend::dellsc_iscsi::iscsi_ip_address: '127.0.0.1'
 cinder::backend::eqlx::eqlx_chap_login: 'user'
 cinder::backend::eqlx::eqlx_chap_password: 'user'
 cinder::backend::emc_vnx::storage_vnx_pool_names: 'emc-storage-pool'
-service_names:
+enabled_services:
  - 'ceilometer_agent_central'
  - 'horizon'
 tripleo_fluentd_groups_ceilometer_agent_central: