From 949982c780abff9fec5ab92946692598c35d26ea Mon Sep 17 00:00:00 2001
From: Monty Taylor <mordred@inaugust.com>
Date: Sun, 27 May 2018 08:48:00 -0500
Subject: [PATCH] Update ansible test job to run against stable-2.5

ansible now depends on openstacksdk and not shade. Update the shade
tests to test against stable-2.5 branch so that we can make sure shade
patches don't break people using older ansible.

Change-Id: Id7af93cbc73b208d7a973ca5101b8977a0ace689
---
 .zuul.yaml | 23 ++++++++++++++++++++++-
 tox.ini    |  2 +-
 2 files changed, 23 insertions(+), 2 deletions(-)

diff --git a/.zuul.yaml b/.zuul.yaml
index 9ec920db4..402840896 100644
--- a/.zuul.yaml
+++ b/.zuul.yaml
@@ -203,6 +203,27 @@
       test_matrix_branch: master
       tox_install_siblings: true
 
+- job:
+    name: shade-ansible-stable-2.5-functional-devstack
+    parent: shade-ansible-functional-devstack
+    description: |
+      Run shade ansible functional tests against a master devstack
+      using git devel branch version of ansible.
+    branches: ^(devel|master)$
+    required-projects:
+      - name: github.com/ansible/ansible
+        override-checkout: stable-2.5
+      - name: openstack-infra/shade
+        override-checkout: master
+      - name: openstack-dev/devstack
+        override-checkout: master
+    vars:
+      # test-matrix grabs branch from the zuul branch setting. If the job
+      # is triggered by ansible, that branch will be stable-2.5 which doesn't
+      # make sense to devstack. Override so that we run the right thing.
+      test_matrix_branch: master
+      tox_install_siblings: true
+
 - project-template:
     name: shade-functional-tips
     check:
@@ -224,7 +245,7 @@
       jobs:
         - bifrost-integration-tinyipa
         - bifrost-integration-tinyipa-opensuse-423
-        - shade-ansible-devel-functional-devstack:
+        - shade-ansible-stable-2.5-functional-devstack:
             voting: false
         - shade-ansible-functional-devstack
         - shade-functional-devstack
diff --git a/tox.ini b/tox.ini
index e1f55d7ec..03c95b35b 100644
--- a/tox.ini
+++ b/tox.ini
@@ -63,7 +63,7 @@ deps =
     -c{env:UPPER_CONSTRAINTS_FILE:https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt}
     -r{toxinidir}/requirements.txt
     -r{toxinidir}/test-requirements.txt
-    ansible
+    ansible<=2.5
 commands = {toxinidir}/extras/run-ansible-tests.sh -e {envdir} {posargs}
 
 [testenv:docs]