From e227681998fbf6f5c545cf1d207f83dc3ed3129b Mon Sep 17 00:00:00 2001
From: Jean-Philippe Evrard <jean-philippe@evrard.me>
Date: Fri, 19 Jan 2018 14:53:29 +0000
Subject: [PATCH] Fix upgrade test

openstacksdk was updated to 0.10.0 which has a different
connection.Connection object, which doesn't take our auth_url.

We should respect the constraints, install the tooling in a venv,
and restrict to a known version of the sdk until more robust code
is written.

Change-Id: I681d5177c3d62d4cd44371fef0032fa8a0727dfd
---
 tests/test-nova-resources-upgrade.yml | 22 ++++++++++++++++++++--
 1 file changed, 20 insertions(+), 2 deletions(-)

diff --git a/tests/test-nova-resources-upgrade.yml b/tests/test-nova-resources-upgrade.yml
index ef2bf4a8..70dec535 100644
--- a/tests/test-nova-resources-upgrade.yml
+++ b/tests/test-nova-resources-upgrade.yml
@@ -21,14 +21,32 @@
   vars:
     test_keystone_host: openstack1
     working_dir: "{{ lookup('env', 'WORKING_DIR') }}"
+    upgrade_venv_folder: "/openstack/venvs/upgrades"
+    upgrade_test_packages:
+      - "python-keystoneclient"
+      - "python-glanceclient"
+      - "openstacksdk"
   tasks:
     - name: Install openstack sdk
       pip:
-        name: "python-openstacksdk"
+        name: "{{ upgrade_test_packages }}"
+        virtualenv: "{{ upgrade_venv_folder }}"
+        virtualenv_site_packages: "no"
+        extra_args: >-
+          {{ (pip_install_upper_constraints is defined) | ternary('--constraint ' + pip_install_upper_constraints | default(''),'') }}
+          {{ pip_install_options | default('') }}
+      register: install_packages
+      until: install_packages | success
+      retries: 5
+      delay: 2
+
     - name: Run rolling test script
-      command: ./openstack-ansible-ops/bowling_ball/rolling_tests.py nova
+      shell: |
+        source {{ upgrade_venv_folder }}/bin/activate
+        ./openstack-ansible-ops/bowling_ball/rolling_tests.py nova
       args:
         chdir: "{{ working_dir }}"
+        executable: /bin/bash
       async: 100000
       poll: 0
       environment: