Skip to content

Commit

Permalink
Split upgrade test into its own play
Browse files Browse the repository at this point in the history
To avoid requiring all the os_previous_swift tasks to be evaluated, we
should split the 2 tests up. This will help make the gate more efficient
and avoids unnecessary issues.

Change-Id: I105d0c49f1b5dc78e1e5a5c7334cd81ecc455056
  • Loading branch information
andymcc committed Jan 20, 2017
1 parent 0c7f419 commit 0e45afb
Show file tree
Hide file tree
Showing 3 changed files with 37 additions and 11 deletions.
32 changes: 32 additions & 0 deletions tests/test-upgrade.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,32 @@
---
# Copyright 2015, Rackspace US, Inc.
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.

# Setup the keys, host and containers
- include: common/test-setup-host.yml

# Install Memcached/Galera/Rabbitmq
- include: common/test-install-infra.yml

# Install Keystone
- include: common/test-install-keystone.yml

# Setup Swift-hosts
- include: common/test-setup-swifthosts.yml

# Install previous swift
- include: test-install-previous-swift.yml

# Install Swift
- include: common/test-install-swift.yml
6 changes: 0 additions & 6 deletions tests/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,11 +25,5 @@
# Setup Swift-hosts
- include: common/test-setup-swifthosts.yml

# Install previous swift
- include: test-install-previous-swift.yml
when:
- swift_upgrade is defined
- swift_upgrade | bool

# Install Swift
- include: common/test-install-swift.yml
10 changes: 5 additions & 5 deletions tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -59,10 +59,6 @@ commands =
bash -c "if [ ! -d "{toxinidir}/tests/common" ]; then \
git clone https://git.openstack.org/openstack/openstack-ansible-tests {toxinidir}/tests/common; \
fi"
bash -c "if [ ! -d "{toxinidir}/tests/common/previous" ]; then \
git clone https://git.openstack.org/openstack/openstack-ansible-tests -b stable/newton \
{toxinidir}/tests/common/previous; \
fi"


[testenv:pep8]
Expand Down Expand Up @@ -119,9 +115,13 @@ deps =
{[testenv:ansible]deps}
setenv =
{[testenv]setenv}
ANSIBLE_PARAMETERS=-vvv -e swift_upgrade=True
TEST_PLAYBOOK={toxinidir}/tests/test-upgrade.yml
commands =
{[testenv:tests_clone]commands}
bash -c "if [ ! -d "{toxinidir}/tests/common/previous" ]; then \
git clone https://git.openstack.org/openstack/openstack-ansible-tests -b stable/newton \
{toxinidir}/tests/common/previous; \
fi"
bash -c "{toxinidir}/tests/common/test-ansible-functional.sh"


Expand Down

0 comments on commit 0e45afb

Please sign in to comment.