openstack/puppet-ironic
Code Issues Proposed changes

Drop redundant package name checks

Browse Source 
ironic-api package and ironic-conductor package are defined for all
supported operating systems, thus these checks are just redundant.

Change-Id: I195a377e22d09aa0ee619f46e07e351dd3b87d35
This commit is contained in:
Takashi Kajinami 2024-06-10 17:42:33 +09:00
parent a7f9c7a358
commit f63900b549
2 changed files with 8 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
manifests/api.pp
View File

@ -102,12 +102,10 @@ class ironic::api (
}
# Install package
if $::ironic::params::api_package {
package { 'ironic-api':
ensure => $package_ensure,
name => $::ironic::params::api_package,
tag => ['openstack', 'ironic-package'],
}
package { 'ironic-api':
ensure => $package_ensure,
name => $::ironic::params::api_package,
tag => ['openstack', 'ironic-package'],
}
if $manage_service {

10
manifests/conductor.pp
View File

@ -375,12 +375,10 @@ class ironic::conductor (
}
# Install package
if $::ironic::params::conductor_package {
package { 'ironic-conductor':
ensure => $package_ensure,
name => $::ironic::params::conductor_package,
tag => ['openstack', 'ironic-package'],
}
package { 'ironic-conductor':
ensure => $package_ensure,
name => $::ironic::params::conductor_package,
tag => ['openstack', 'ironic-package'],
}
if $manage_service {