]> review.fuel-infra Code Review - puppet-modules/puppet-ceilometer.git/commitdiff
Remove parameters for [hardware] options
authorTakashi Kajinami <tkajinam@redhat.com>
Tue, 15 Nov 2022 02:13:31 +0000 (11:13 +0900)
committerTakashi Kajinami <tkajinam@redhat.com>
Tue, 15 Nov 2022 02:15:50 +0000 (11:15 +0900)
... because these parameters were deprecated during the previous
cycle[1] and the feature has been removed from Ceilometer already.

[1] e5ce40b16d7534a917416cb2e5f3669336fc816f

Change-Id: Ibaba550aa9a13c7b1afb62970e391c1d4beb2f08

manifests/init.pp
releasenotes/notes/remove-hardware-opts-88e4b49767d7b185.yaml [new file with mode: 0644]

index f890e7508936b3947856634b2c986c5b7b27eca2..c374025dba91cd8f66a3dfb82149a739fa18c5d6 100644 (file)
 #   IP address.
 #   Defaults to $::os_service_default.
 #
-# DEPRECATED PARAMETERS
-#
-# [*snmpd_readonly_username*]
-#   (Optional) User name for snmpd authentication
-#   Defaults to undef
-#
-# [*snmpd_readonly_user_password*]
-#   (Optional) Password for snmpd authentication
-#   Defaults to undef
-#
 class ceilometer(
   $http_timeout                       = $::os_service_default,
   $max_parallel_requests              = $::os_service_default,
@@ -263,18 +253,11 @@ class ceilometer(
   $amqp_password                      = $::os_service_default,
   $purge_config                       = false,
   $host                               = $::os_service_default,
-  # DEPRECATED PARAMETERS
-  $snmpd_readonly_username            = undef,
-  $snmpd_readonly_user_password       = undef,
 ) {
 
   include ceilometer::deps
   include ceilometer::params
 
-  if $snmpd_readonly_username != undef or $snmpd_readonly_user_password != undef {
-    warning('The snmpd_readonly_* parameters have been deprecated and have no effect.')
-  }
-
   package { 'ceilometer-common':
     ensure => $package_ensure,
     name   => $::ceilometer::params::common_package_name,
@@ -329,11 +312,6 @@ class ceilometer(
     'DEFAULT/host'                 : value => $host;
     'publisher/telemetry_secret'   : value => $telemetry_secret, secret => true;
   }
-  # TODO(tkajinam): Remove this after Zed
-  ceilometer_config {
-    'hardware/readonly_user_name'    : ensure => absent;
-    'hardware/readonly_user_password': ensure => absent, secret => true;
-  }
 
   oslo::messaging::notifications { 'ceilometer_config':
     transport_url => $notification_transport_url,
diff --git a/releasenotes/notes/remove-hardware-opts-88e4b49767d7b185.yaml b/releasenotes/notes/remove-hardware-opts-88e4b49767d7b185.yaml
new file mode 100644 (file)
index 0000000..e210d39
--- /dev/null
@@ -0,0 +1,8 @@
+---
+upgrade:
+  - |
+    The following parameters of the ``ceilometer`` class have been removed.
+
+    - ``snmpd_readonly_user_name``
+    - ``snmpd_readonly_user_password``
+