setup.py: Remove 'py_modules'

This was merely obscuring a bug in pbr. setuptools doesn't do the
auto-discovery when pbr is in use. Remove it.

Also fixes the environments list where it is now validated for what is
installed by removing functional-py38, and fixing the job to just be
"functional".

Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
Depends-on: https://review.opendev.org/c/openstack/pbr/+/869082
Change-Id: Ife999840777f3c553f569d81cd826ef9d5a6765d

setup.py

@@ -17,5 +17,5 @@ import setuptools
 
 setuptools.setup(
     setup_requires=['pbr>=2.0.0'],
-    py_modules=[],
+    pbr=True,
-    pbr=True)
+)

tox.ini

@@ -1,6 +1,6 @@
 [tox]
 minversion = 3.18.0
-envlist = py3,pep8,functional-py38
+envlist = py3,pep8
 ignore_basepython_conflict=true
 
 [testenv]
@@ -60,7 +60,7 @@ commands =
     flake8 ironic_inspector
     doc8 -e .rst -e .inc README.rst CONTRIBUTING.rst doc/source api-ref/source
 
-[testenv:functional-py38]
+[testenv:functional]
 commands =
     python3 -m ironic_inspector.test.functional {posargs}
 

zuul.d/project.yaml

@@ -14,7 +14,7 @@
         #to fix right after
         - ironic-inspector-tempest-managed-non-standalone:
             voting: false
-        - openstack-tox-functional-py38
+        - openstack-tox-functional
         - bifrost-integration-tinyipa-ubuntu-focal:
             voting: false
         - ironic-inspector-tox-bandit:
@@ -24,4 +24,4 @@
         - ironic-inspector-grenade
         - ironic-inspector-tempest
         - ironic-inspector-tempest-discovery
-        - openstack-tox-functional-py38
+        - openstack-tox-functional