]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph-build.git/commitdiff
ceph-ansible-nightly/build: Simplify run_tox() logic 986/head
authorErwan Velu <erwan@redhat.com>
Fri, 16 Mar 2018 12:02:52 +0000 (13:02 +0100)
committerErwan Velu <erwan@redhat.com>
Fri, 16 Mar 2018 12:02:52 +0000 (13:02 +0100)
The actual logic was a little bit complicated to read.
Since find_latest_tag() is generic, let's reuse it inside a case() loop.

ceph-ansible-nightly/build/build

index 8259930ac518587633dd2862719f0399e67dc8e6..964e3e9cd7e525d1f5190b37617c3cc25241ef32 100644 (file)
@@ -31,17 +31,21 @@ function find_latest_tag {
 }
 
 function run_tox {
-  if [ "$RELEASE" == 'jewel' ] && [ "$CEPH_ANSIBLE_BRANCH" == 'stable-2.2' -o "$CEPH_ANSIBLE_BRANCH" == 'stable-3.0' ]; then
-    start_tox CEPH_DOCKER_IMAGE_TAG="$LAST_JEWEL_STABLE_TAG"
-  elif [ "$RELEASE" == 'luminous' ] && [ "$CEPH_ANSIBLE_BRANCH" == 'stable-3.0' ]; then
-    # start_tox(): <CEPH_DOCKER_IMAGE_TAG> <CEPH_STABLE_RELEASE>
-    start_tox CEPH_DOCKER_IMAGE_TAG="$LAST_LUMINOUS_STABLE_TAG"
-  elif [ "$RELEASE" == 'luminous' ] && [ "$CEPH_ANSIBLE_BRANCH" == 'master' ]; then
-    start_tox CEPH_DOCKER_IMAGE_TAG=latest
-  fi
+  case "$CEPH_ANSIBLE_BRANCH" in
+    stable-*)
+      CEPH_DOCKER_IMAGE_TAG=$(find_latest_tag "$RELEASE")
+      ;;
+    master)
+      CEPH_DOCKER_IMAGE_TAG="latest"
+      ;;
+    *)
+      CEPH_DOCKER_IMAGE_TAG=""
+      ;;
+  esac
+
+  [ -n "$CEPH_DOCKER_IMAGE_TAG" ] && start_tox CEPH_DOCKER_IMAGE_TAG="$CEPH_DOCKER_IMAGE_TAG"
 }
 
-
 ########
 # MAIN #
 ########
@@ -50,6 +54,4 @@ clear_libvirt_networks
 restart_libvirt_services
 update_vagrant_boxes
 count_tag_pages
-LAST_JEWEL_STABLE_TAG=$(find_latest_tag jewel)
-LAST_LUMINOUS_STABLE_TAG=$(find_latest_tag luminous)
 run_tox