From 0a2c1ff8089b29f7d600ab358d2727b52d35a3d4 Mon Sep 17 00:00:00 2001 From: Douglas Henrique Koerich Date: Tue, 17 Aug 2021 13:04:28 -0300 Subject: [PATCH] FEC devices enabled by default in puppet manifest In the context of making pf-bb-config tool being launched to configure also N3000 devices (https://storyboard.openstack.org/#!/story/2009036) the puppet manifest relies on the 'enabled' parameter which is set via a sysinv script based on PCI devices inventory. It was observed that if this feature is available not by a fresh install but instead through a patch applied on the corresponding modules, it required an extra reboot to have the hieradata populated with that new parameter, which is unde- sired in installed systems. This change makes the puppet manifest to consider the FEC devices are enabled by default. By doing that, the first run of the manifest after applying the patch won't rely on the (absent) hieradata parameter, but will still trigger the launch of pf-bb-config tool to configure the FEC device. Later on, it is expected that 'enabled' parameter is added to hieradata and manifest can rely on that for any further configuration. NOTE: actually, this change makes puppet manifest to match the same behaviour of the corresponding at database. There, 'enabled' field for devices is 'True' by default. Task: 43030 Story: 2009036 Signed-off-by: Douglas Henrique Koerich Change-Id: I65d1c6cc34029b20a4777dbdafa0cddf04082206 --- puppet-manifests/src/modules/platform/manifests/devices.pp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/puppet-manifests/src/modules/platform/manifests/devices.pp b/puppet-manifests/src/modules/platform/manifests/devices.pp index 888854f4..5cc82969 100644 --- a/puppet-manifests/src/modules/platform/manifests/devices.pp +++ b/puppet-manifests/src/modules/platform/manifests/devices.pp @@ -164,7 +164,7 @@ class platform::devices::fpga::fec { } class platform::devices::n3000::fec ( - $enabled = false + $enabled = true ) {} class platform::devices::n3000::config @@ -178,7 +178,7 @@ class platform::devices::n3000::config } class platform::devices::acc100::fec ( - $enabled = false + $enabled = true ) {} class platform::devices::acc100::config (