From ca56dacc6d01a16227843444fe10d01f6cbb716e Mon Sep 17 00:00:00 2001 From: Alexandru Avadanii Date: Sun, 20 Aug 2017 17:57:08 +0200 Subject: maas: Refactor boot resources infinite loops Change-Id: I4c1322e2ddd96ec5a1f4b11ece60b635f7e043c5 Signed-off-by: Alexandru Avadanii --- ...-Break-infinite-loops-into-finite-retries.patch | 67 ++++++++++++++++++++++ 1 file changed, 67 insertions(+) create mode 100644 patches/opnfv-fuel/0000-maas-Break-infinite-loops-into-finite-retries.patch (limited to 'patches/opnfv-fuel/0000-maas-Break-infinite-loops-into-finite-retries.patch') diff --git a/patches/opnfv-fuel/0000-maas-Break-infinite-loops-into-finite-retries.patch b/patches/opnfv-fuel/0000-maas-Break-infinite-loops-into-finite-retries.patch new file mode 100644 index 00000000..7b8ce8f6 --- /dev/null +++ b/patches/opnfv-fuel/0000-maas-Break-infinite-loops-into-finite-retries.patch @@ -0,0 +1,67 @@ +From: Alexandru Avadanii +Date: Sun, 20 Aug 2017 17:52:56 +0200 +Subject: [PATCH] maas: Break infinite loops into finite retries + +While at it, move the bash commands to a separate script file. + +Signed-off-by: Alexandru Avadanii +--- + .../0003-maas-region-force-artifact-download.patch | 35 +++++++++++++++++++--- + 1 file changed, 31 insertions(+), 4 deletions(-) + +diff --git a/mcp/patches/0003-maas-region-force-artifact-download.patch b/mcp/patches/0003-maas-region-force-artifact-download.patch +index 646bed3..f0ce50f 100644 +--- a/mcp/patches/0003-maas-region-force-artifact-download.patch ++++ b/mcp/patches/0003-maas-region-force-artifact-download.patch +@@ -15,16 +15,17 @@ Signed-off-by: Alexandru Avadanii + --- + + diff --git a/maas/region.sls b/maas/region.sls +-index d3227ca..8a2243d 100644 + --- a/maas/region.sls + +++ b/maas/region.sls +-@@ -109,11 +109,17 @@ ++@@ -109,11 +109,19 @@ + cmd.run: + - name: "maas-region apikey --username {{ region.admin.username }} > /var/lib/maas/.maas_credentials" + + +maas_force_artifact_sync: +-+ cmd.run: +-+ - name: "maas login {{ region.admin.username }} http://{{ region.bind.host }}:5240/MAAS/api/2.0 - < /var/lib/maas/.maas_credentials && while ! grep -qzE '(Unable to probe for DHCP servers|DHCP probe complete).*Rack controller' /var/log/maas/rackd.log; do sleep 5; echo -n '.'; done && maas opnfv boot-resources import && while maas opnfv boot-resources is-importing | grep -q -e 'true'; do sleep 5; echo -n '.'; done && maas opnfv rack-controllers import-boot-images && while ! test -d /var/lib/maas/boot-resources/current/ubuntu; do sleep 5; echo -n '.'; done" +++ cmd.script: +++ - name: salt://maas/files/maas-artifact-sync.sh +++ - template: jinja +++ - shell: /bin/bash + + - require: + + - cmd: maas_login_admin + + +@@ -37,3 +38,29 @@ index d3227ca..8a2243d 100644 + + maas_commissioning_scripts: + module.run: ++diff --git a/maas/files/maas-artifact-sync.sh b/maas/files/maas-artifact-sync.sh ++new file mode 100644 ++--- /dev/null +++++ b/maas/files/maas-artifact-sync.sh ++@@ -0,0 +1,21 @@ +++{%- from "maas/map.jinja" import region with context %} +++#!/bin/bash +++function wait_for { +++ local total_attempts=$1; shift +++ local cmdstr=$@ +++ local sleep_time=10 +++ echo -e "\n[NOTE] Waiting for cmd to return success: ${cmdstr}\n" +++ for attempt in $(seq "${total_attempts}"); do +++ eval "${cmdstr}" && break || true +++ echo -n '.'; sleep "${sleep_time}" +++ done +++} +++maas login {{ region.admin.username }} \ +++ http://{{ region.bind.host }}:5240/MAAS/api/2.0 - < \ +++ /var/lib/maas/.maas_credentials || exit 1 +++# wait max 15 min for service up / image download, 5 min region to rack sync +++wait_for 90 "grep -qzE '(Unable to probe for DHCP servers|DHCP probe complete).*Rack controller' /var/log/maas/rackd.log" +++maas opnfv boot-resources import || exit 2 +++wait_for 90 "! maas opnfv boot-resources is-importing | grep -q 'true'" +++maas opnfv rack-controllers import-boot-images || exit 3 +++wait_for 30 "test -d /var/lib/maas/boot-resources/current/ubuntu/amd64" -- cgit 1.2.3-korg