From 4f1bfdd657da7f8daa928e79041405de86993d8f Mon Sep 17 00:00:00 2001 From: Alexandru Avadanii Date: Thu, 24 Aug 2017 19:18:30 +0200 Subject: p/fuel: states: maas: Batch apply linux state Change-Id: I62f96c93b67c79732dc8737ff5746475eb5259a6 Signed-off-by: Alexandru Avadanii --- ...-config-AArch64-baremetal-Use-arm64-image.patch | 2 +- ...twork-public-mgmt-Use-arm-pod3-POD-config.patch | 2 +- ...lt-master-Increase-command_timeout-to-15s.patch | 38 -------------------- ...tes-maas-Batch-apply-linux-state-with-b-4.patch | 42 ++++++++++++++++++++++ ...-maas-rm-Ubuntu-boot-entry-on-EFI-systems.patch | 2 +- 5 files changed, 45 insertions(+), 41 deletions(-) delete mode 100644 patches/opnfv-fuel/0018-salt-master-Increase-command_timeout-to-15s.patch create mode 100644 patches/opnfv-fuel/0018-states-maas-Batch-apply-linux-state-with-b-4.patch diff --git a/patches/opnfv-fuel/0007-mcp-config-AArch64-baremetal-Use-arm64-image.patch b/patches/opnfv-fuel/0007-mcp-config-AArch64-baremetal-Use-arm64-image.patch index e238250c..26b0dc33 100644 --- a/patches/opnfv-fuel/0007-mcp-config-AArch64-baremetal-Use-arm64-image.patch +++ b/patches/opnfv-fuel/0007-mcp-config-AArch64-baremetal-Use-arm64-image.patch @@ -25,7 +25,7 @@ index b841e88..17fbbfc 100644 ram: 4096 diff --git a/mcp/reclass/classes/cluster/baremetal-mcp-ocata-odl-ha/infra/init.yml b/mcp/reclass/classes/cluster/baremetal-mcp-ocata-odl-ha/infra/init.yml -index 5862ddb..ab14c58 100644 +index 5862ddb..2374dee 100644 --- a/mcp/reclass/classes/cluster/baremetal-mcp-ocata-odl-ha/infra/init.yml +++ b/mcp/reclass/classes/cluster/baremetal-mcp-ocata-odl-ha/infra/init.yml @@ -51,7 +51,7 @@ parameters: diff --git a/patches/opnfv-fuel/0010-network-public-mgmt-Use-arm-pod3-POD-config.patch b/patches/opnfv-fuel/0010-network-public-mgmt-Use-arm-pod3-POD-config.patch index 4a159446..5bbe783a 100644 --- a/patches/opnfv-fuel/0010-network-public-mgmt-Use-arm-pod3-POD-config.patch +++ b/patches/opnfv-fuel/0010-network-public-mgmt-Use-arm-pod3-POD-config.patch @@ -41,7 +41,7 @@ index 1e14e5f..db0241b 100755 wait_for 10 "salt -C 'mas01*' state.apply maas.region" diff --git a/mcp/reclass/classes/cluster/baremetal-mcp-ocata-odl-ha/infra/init.yml b/mcp/reclass/classes/cluster/baremetal-mcp-ocata-odl-ha/infra/init.yml -index ab14c58..25746b5 100644 +index 2374dee..fb5b856 100644 --- a/mcp/reclass/classes/cluster/baremetal-mcp-ocata-odl-ha/infra/init.yml +++ b/mcp/reclass/classes/cluster/baremetal-mcp-ocata-odl-ha/infra/init.yml @@ -16,19 +16,19 @@ parameters: diff --git a/patches/opnfv-fuel/0018-salt-master-Increase-command_timeout-to-15s.patch b/patches/opnfv-fuel/0018-salt-master-Increase-command_timeout-to-15s.patch deleted file mode 100644 index 3380de0a..00000000 --- a/patches/opnfv-fuel/0018-salt-master-Increase-command_timeout-to-15s.patch +++ /dev/null @@ -1,38 +0,0 @@ -From: Alexandru Avadanii -Date: Thu, 24 Aug 2017 03:50:11 +0200 -Subject: [PATCH] salt-master: Increase command_timeout to 15s - -Default command timeout (10s) is sometimes too small for all minions -to return within the required timeslot. Increase it by 50%. - -While at it, increase the timeout used for executing the "linux" -state against all nodes to 60s, just to be on the safe side. - -Signed-off-by: Alexandru Avadanii ---- - mcp/config/states/maas | 2 +- - mcp/reclass/classes/cluster/baremetal-mcp-ocata-ovs-ha/infra/config.yml | 1 + - 2 files changed, 2 insertions(+), 1 deletion(-) - -diff --git a/mcp/config/states/maas b/mcp/config/states/maas -index 12ef9ed..14f32ee 100755 ---- a/mcp/config/states/maas -+++ b/mcp/config/states/maas -@@ -107,4 +107,4 @@ wait_for 10 "salt -C '* and not cfg01* and not mas01*' ssh.set_auth_key ${SUDO_U - - wait_for 10 "salt -C '* and not cfg01* and not mas01*' saltutil.sync_all" - wait_for 10 "salt -C '* and not cfg01* and not mas01*' state.apply salt" --wait_for 10 "salt -C '* and not cfg01* and not mas01*' state.apply linux,ntp" -+wait_for 10 "salt -C '* and not cfg01* and not mas01*' state.apply linux,ntp -t 60" -diff --git a/mcp/reclass/classes/cluster/baremetal-mcp-ocata-ovs-ha/infra/config.yml b/mcp/reclass/classes/cluster/baremetal-mcp-ocata-ovs-ha/infra/config.yml -index 7d95ebc..46526da 100644 ---- a/mcp/reclass/classes/cluster/baremetal-mcp-ocata-ovs-ha/infra/config.yml -+++ b/mcp/reclass/classes/cluster/baremetal-mcp-ocata-ovs-ha/infra/config.yml -@@ -37,6 +37,7 @@ parameters: - salt: - master: - accept_policy: open_mode -+ command_timeout: 15 - reclass: - storage: - data_source: diff --git a/patches/opnfv-fuel/0018-states-maas-Batch-apply-linux-state-with-b-4.patch b/patches/opnfv-fuel/0018-states-maas-Batch-apply-linux-state-with-b-4.patch new file mode 100644 index 00000000..3f4986b4 --- /dev/null +++ b/patches/opnfv-fuel/0018-states-maas-Batch-apply-linux-state-with-b-4.patch @@ -0,0 +1,42 @@ +From: Alexandru Avadanii +Date: Thu, 24 Aug 2017 03:50:11 +0200 +Subject: [PATCH] states: maas: Batch apply linux state with `-b 4` + +Applying heavy states like linux.network on multiple nodes in +parallel might lead to spurious timeouts on slower systems. + +Compromise between speed and realiability by applying this state +in batches of 4 nodes at once. + +While at it, increase global command_timeout to 15s on Salt master, +since default command timeout (10s) is sometimes too small for all +minions to return within the required timeslot. Increase it by 50%. + +Signed-off-by: Alexandru Avadanii +--- + mcp/config/states/maas | 2 +- + mcp/reclass/classes/cluster/baremetal-mcp-ocata-ovs-ha/infra/config.yml | 1 + + 2 files changed, 2 insertions(+), 1 deletion(-) + +diff --git a/mcp/config/states/maas b/mcp/config/states/maas +index 12ef9ed..697f83c 100755 +--- a/mcp/config/states/maas ++++ b/mcp/config/states/maas +@@ -107,4 +107,4 @@ wait_for 10 "salt -C '* and not cfg01* and not mas01*' ssh.set_auth_key ${SUDO_U + + wait_for 10 "salt -C '* and not cfg01* and not mas01*' saltutil.sync_all" + wait_for 10 "salt -C '* and not cfg01* and not mas01*' state.apply salt" +-wait_for 10 "salt -C '* and not cfg01* and not mas01*' state.apply linux,ntp" ++wait_for 10 "salt -C '* and not cfg01* and not mas01*' state.apply linux,ntp -b 4" +diff --git a/mcp/reclass/classes/cluster/baremetal-mcp-ocata-ovs-ha/infra/config.yml b/mcp/reclass/classes/cluster/baremetal-mcp-ocata-ovs-ha/infra/config.yml +index 7d95ebc..46526da 100644 +--- a/mcp/reclass/classes/cluster/baremetal-mcp-ocata-ovs-ha/infra/config.yml ++++ b/mcp/reclass/classes/cluster/baremetal-mcp-ocata-ovs-ha/infra/config.yml +@@ -37,6 +37,7 @@ parameters: + salt: + master: + accept_policy: open_mode ++ command_timeout: 15 + reclass: + storage: + data_source: diff --git a/patches/opnfv-fuel/0019-states-maas-rm-Ubuntu-boot-entry-on-EFI-systems.patch b/patches/opnfv-fuel/0019-states-maas-rm-Ubuntu-boot-entry-on-EFI-systems.patch index 792774f3..cc9be244 100644 --- a/patches/opnfv-fuel/0019-states-maas-rm-Ubuntu-boot-entry-on-EFI-systems.patch +++ b/patches/opnfv-fuel/0019-states-maas-rm-Ubuntu-boot-entry-on-EFI-systems.patch @@ -24,7 +24,7 @@ Signed-off-by: Alexandru Avadanii 1 file changed, 5 insertions(+) diff --git a/mcp/config/states/maas b/mcp/config/states/maas -index 14f32ee..b9f3ac6 100755 +index 697f83c..383063d 100755 --- a/mcp/config/states/maas +++ b/mcp/config/states/maas @@ -44,6 +44,11 @@ function maas_fixup() { -- cgit 1.2.3-korg