openstack/puppet-nova
Code Issues Proposed changes

Remove redundant usage of pick function

Browse Source 
The cpu_shared_set parameter and the cpu_dedicated_set parameter are
set to $::os_service_default by default, so we don't need to implement
a safe default by pick function.

Change-Id: I69fe5993232a8b37e4b9f527548932c6a2f83864
This commit is contained in:
Takashi Kajinami
2021-01-12 13:13:21 +09:00
parent 67ab0e4ee2
commit c89593bbef
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
manifests/compute.pp
View File

@@ -359,8 +359,8 @@ class nova::compute (
include nova::deps
include nova::params
$cpu_shared_set_real = pick(join(any2array($cpu_shared_set), ','), $::os_service_default)
$cpu_dedicated_set_real = pick(join(any2array($cpu_dedicated_set), ','), $::os_service_default)
$cpu_shared_set_real = join(any2array($cpu_shared_set), ',')
$cpu_dedicated_set_real = join(any2array($cpu_dedicated_set), ',')
$image_type_exclude_list_real = pick(join(any2array($image_type_exclude_list), ','), $::os_service_default)