Fix travis (#10)

* Use py3 for tox by default, use train UC for python <= 3.5

Upstream OpenStack has dropped support for python 2, and even python
3.5. This means that master upper constraints no longer install on these
versions.

* Test with python 3.8

* Test with python3.6 and python3.7

3.8 isn't a supported runtime for OpenStack until Victoria. Train and Ussuri support 3.6-3.7.

* Workaround for python3.8

* Whitespace fix

Co-authored-by: Mark Goddard <[email protected]>

Author: jovial

.travis.yml

@@ -1,6 +1,5 @@
 ---
 language: python
-python: "2.7"
 
 # Run jobs in containers.
 sudo: false
@@ -14,16 +13,26 @@ addons:
       - realpath
 
 # Create a build matrix for the different test jobs.
-env:
-  matrix:
-    # Run python style checks.
-    - TOX_ENV=pep8
+matrix:
+  include:
     # Run python2.7 unit tests.
-    - TOX_ENV=py27
+    - python: 2.7
+      env: TOXENV=py27
+    # Run python style checks.
+    - python: 3.8
+      env: TOXENV=pep8
+    # Run python3.6 unit tests.
+    - python: 3.6
+      env: TOXENV=py36
+    # Run python3.7 unit tests.
+    - python: 3.7
+      env: TOXENV=py37
+    # Run python3.8 unit tests.
+    - python: 3.8
+      env: TOXENV=py38
 
 install:
   - pip install tox
 
 script:
-  # Run the tox environment.
-  - tox -e ${TOX_ENV}
+  - tox

test-requirements.txt

@@ -1,6 +1,8 @@
 # The order of packages is significant, because pip processes them in the order
 # of appearance. Changing the order has an impact on the overall integration
 # process, which may cause wedges in the gate later.
+pyflakes>=2.2.0 # MIT
+python-ironicclient # Apache-2.0
 coverage!=4.4,>=4.0 # Apache-2.0
 flake8-import-order==0.11 # LGPLv3
 hacking<0.13,>=0.12.0 # Apache-2.0

tox.ini

@@ -1,10 +1,13 @@
 [tox]
-envlist = py35,py27,pep8
+envlist = py36,py37,py38,py27,pep8
 
 [testenv]
 usedevelop = True
-install_command = pip install -U -c{env:UPPER_CONSTRAINTS_FILE:https://git.openstack.org/cgit/openstack/requirements/plain/upper-constraints.txt} {opts} {packages}
+install_command = pip install -U {opts} {packages}
+basepython = python3.8
 deps =
+    -c{env:UPPER_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master}
+    -r{toxinidir}/requirements.txt
     -r{toxinidir}/test-requirements.txt
 commands =
     coverage run --branch --include "stackhpc_inspector_plugins*" -m unittest discover stackhpc_inspector_plugins.tests.unit
@@ -22,10 +25,38 @@ commands =
     coverage report -m
 
 [testenv:pep8]
-basepython = python2.7
 commands =
     flake8 stackhpc_inspector_plugins
 
+[testenv:py27]
+basepython = python2.7
+deps =
+    -c{env:UPPER_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/train}
+    -r{toxinidir}/requirements.txt
+    -r{toxinidir}/test-requirements.txt
+
+[testenv:py38]
+basepython = python3.8
+# FIXME: Use ussuri release until victoria is released.
+deps =
+    -c{env:UPPER_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/ussuri}
+    -r{toxinidir}/requirements.txt
+    -r{toxinidir}/test-requirements.txt
+
+[testenv:py37]
+basepython = python3.7
+deps =
+    -c{env:UPPER_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/ussuri}
+    -r{toxinidir}/requirements.txt
+    -r{toxinidir}/test-requirements.txt
+
+[testenv:py36]
+basepython = python3.6
+deps =
+    -c{env:UPPER_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/train}
+    -r{toxinidir}/requirements.txt
+    -r{toxinidir}/test-requirements.txt
+
 [flake8]
 max-complexity=15
 # [H106] Don’t put vim configuration in source files.