Fix OS_CLOUD export for image validation 18/62318/2 v0.48.1
authorAndrew Grimberg <agrimberg@linuxfoundation.org>
Mon, 18 Nov 2019 17:47:44 +0000 (09:47 -0800)
committerAndrew Grimberg <agrimberg@linuxfoundation.org>
Mon, 18 Nov 2019 17:55:20 +0000 (09:55 -0800)
Change fe2beb15bc9effb5b91ba17b520ab6c167dd6604 introduced a bug where
the CLOUD_CREDENTIAL_ID was being used for the OS_CLOUD export variable.
This was done since the test for multi-cloud image validation had the
credential IDs matching the OS_CLOUD variable. However, most of our
systems have the CLOUD_CREDENTIAL_ID set to the default of 'os-cloud'
and therefore after pulling updating to a global-jjb with that change
would no longer be able to validate images!

We now look for an OS_CLOUD variable in the cloud.cfg file and use it if
it is set, otherwise we default to an OS_CLOUD of 'vex' as we previously
did before change fe2beb15bc9effb5b91ba17b520ab6c167dd6604

Change-Id: I0af763495fa3a7ce2e8c080324abaf5d384e5188
Signed-off-by: Andrew Grimberg <agrimberg@linuxfoundation.org>
releasenotes/notes/fix-multi-cloud-validator-dedb41f09bb82a81.yaml [new file with mode: 0644]
shell/jenkins-verify-images.sh

diff --git a/releasenotes/notes/fix-multi-cloud-validator-dedb41f09bb82a81.yaml b/releasenotes/notes/fix-multi-cloud-validator-dedb41f09bb82a81.yaml
new file mode 100644 (file)
index 0000000..cfca7ba
--- /dev/null
@@ -0,0 +1,6 @@
+---
+fixes:
+  - |
+    Change multi-cloud image validation to look for an OS_CLOUD variable in the
+    cloud.cfg instead of overloading CLOUD_CREDENTIAL_ID as that variable
+    does not usually point to an ID that is in the openstack clouds.yaml file.
index f05e3c3..e4e9e70 100755 (executable)
@@ -44,7 +44,8 @@ for cloud in jenkins-config/clouds/openstack/*; do
   # Verify that we have a cloud config file
   if [ -f "$cloud/cloud.cfg" ]; then
     # Get the OS_CLOUD variable from cloud config and export it
-    OS_CLOUD=$(grep ^CLOUD_CREDENTIAL_ID= "$cloud/cloud.cfg" | cut -d'=' -f2)
+    os_cloud=$(grep ^OS_CLOUD= "$cloud/cloud.cfg" | cut -d'=' -f2)
+    OS_CLOUD="${os_cloud:-vex}"
     export OS_CLOUD
 
     verify_images "$cloud"