summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAlexandru Avadanii <Alexandru.Avadanii@enea.com>2017-08-20 17:57:08 +0200
committerAlexandru Avadanii <Alexandru.Avadanii@enea.com>2017-08-20 18:01:06 +0200
commitca56dacc6d01a16227843444fe10d01f6cbb716e (patch)
tree7b395c75b523bc6887881fcb6f74c1112771c317
parent58da71c2a4c42a354462ee8c6e332c98c1560a58 (diff)
maas: Refactor boot resources infinite loops
Change-Id: I4c1322e2ddd96ec5a1f4b11ece60b635f7e043c5 Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
-rw-r--r--patches/opnfv-fuel/0000-maas-Break-infinite-loops-into-finite-retries.patch67
-rw-r--r--patches/opnfv-fuel/0013-maas-boot-resources-Add-arm64-architecture.patch33
2 files changed, 88 insertions, 12 deletions
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 <Alexandru.Avadanii@enea.com>
+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 <Alexandru.Avadanii@enea.com>
+---
+ .../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 <Alexandru.Avadanii@enea.com>
+ ---
+
+ 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"
diff --git a/patches/opnfv-fuel/0013-maas-boot-resources-Add-arm64-architecture.patch b/patches/opnfv-fuel/0013-maas-boot-resources-Add-arm64-architecture.patch
index bb77f974..517dd232 100644
--- a/patches/opnfv-fuel/0013-maas-boot-resources-Add-arm64-architecture.patch
+++ b/patches/opnfv-fuel/0013-maas-boot-resources-Add-arm64-architecture.patch
@@ -4,19 +4,28 @@ Subject: [PATCH] maas: boot-resources: Add arm64 architecture
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
---
- mcp/patches/0003-maas-region-force-artifact-download.patch | 2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
+ mcp/patches/0003-maas-region-force-artifact-download.patch | 7 ++++++-
+ 1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/mcp/patches/0003-maas-region-force-artifact-download.patch b/mcp/patches/0003-maas-region-force-artifact-download.patch
-index 646bed3..ca61e5e 100644
+index f0ce50f..9997552 100644
--- a/mcp/patches/0003-maas-region-force-artifact-download.patch
+++ b/mcp/patches/0003-maas-region-force-artifact-download.patch
-@@ -24,7 +24,7 @@ index d3227ca..8a2243d 100644
-
- +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"
-++ - 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 && maas opnfv boot-source-selection update 1 1 arches='amd64' arches='arm64' && 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/arm64; do sleep 5; echo -n '.'; done"
- + - require:
- + - cmd: maas_login_admin
- +
+@@ -42,7 +42,7 @@ 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 @@
++@@ -0,0 +1,26 @@
+ +{%- from "maas/map.jinja" import region with context %}
+ +#!/bin/bash
+ +function wait_for {
+@@ -64,3 +64,8 @@ new file mode 100644
+ +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"
+++maas opnfv boot-source-selection update 1 1 arches='amd64' arches='arm64' || exit 4
+++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/arm64"