Merge "pre-commit: Add mypy"

Author: Zuul

.pre-commit-config.yaml

@@ -40,4 +40,38 @@ repos:
     rev: v2.2.4
     hooks:
     - id: codespell
-      args: ['--ignore-words=doc/dictionary.txt']
+      args: ['--ignore-words=doc/dictionary.txt']
+  - repo: https://github.com/pre-commit/mirrors-mypy
+    rev: v1.4.1
+    hooks:
+      - id: mypy
+        additional_dependencies:
+          - types-paramiko
+        # keep this in-sync with '[mypy] files' in 'setup.cfg'
+        files: |
+          (?x)(
+            nova/compute/manager.py
+            | nova/compute/pci_placement_translator.py
+            | nova/crypto.py
+            | nova/filesystem.py
+            | nova/limit/local.py
+            | nova/limit/placement.py
+            | nova/network/neutron.py
+            | nova/pci
+            | nova/privsep/path.py
+            | nova/scheduler/client/report.py
+            | nova/scheduler/request_filter.py
+            | nova/scheduler/utils.py
+            | nova/virt/driver.py
+            | nova/virt/hardware.py
+            | nova/virt/libvirt/machine_type_utils.py
+            | nova/virt/libvirt/__init__.py
+            | nova/virt/libvirt/cpu/__init__.py
+            | nova/virt/libvirt/cpu/api.py
+            | nova/virt/libvirt/cpu/core.py
+            | nova/virt/libvirt/driver.py
+            | nova/virt/libvirt/event.py
+            | nova/virt/libvirt/guest.py
+            | nova/virt/libvirt/host.py
+            | nova/virt/libvirt/utils.py
+          )

mypy-files.txt

@@ -101,8 +101,33 @@ follow_imports = skip
 incremental = true
 check_untyped_defs = true
 warn_unused_ignores = true
+files =
+  nova/compute/manager.py,
+  nova/compute/pci_placement_translator.py,
+  nova/crypto.py,
+  nova/filesystem.py,
+  nova/limit/local.py,
+  nova/limit/placement.py,
+  nova/network/neutron.py,
+  nova/pci/*.py,
+  nova/privsep/path.py,
+  nova/scheduler/client/report.py,
+  nova/scheduler/request_filter.py,
+  nova/scheduler/utils.py,
+  nova/virt/driver.py,
+  nova/virt/hardware.py,
+  nova/virt/libvirt/machine_type_utils.py,
+  nova/virt/libvirt/__init__.py,
+  nova/virt/libvirt/cpu/__init__.py,
+  nova/virt/libvirt/cpu/api.py,
+  nova/virt/libvirt/cpu/core.py,
+  nova/virt/libvirt/driver.py,
+  nova/virt/libvirt/event.py,
+  nova/virt/libvirt/guest.py,
+  nova/virt/libvirt/host.py,
+  nova/virt/libvirt/utils.py
 
 [codespell]
 skip = *.po,*.js,*.css,*.html,*.svg,HACKING.py,*hacking*,*build*,*_static*,doc/dictionary.txt,*.pyc,*.inv,*.gz,*.jpg,*.png,*.vsd,*.graffle,*.json
 count =
-quiet-level = 4
+quiet-level = 4

setup.cfg

@@ -1,6 +1,4 @@
 hacking==6.1.0 # Apache-2.0
-mypy>=0.761 # MIT
-types-paramiko>=0.1.3 # Apache-2.0
 coverage!=4.4,>=4.0 # Apache-2.0
 ddt>=1.2.1 # MIT
 fixtures>=3.0.0 # Apache-2.0/BSD

test-requirements.txt

@@ -99,22 +99,23 @@ extras =
 commands =
   stestr --test-path=./nova/tests/functional run --exclude-regex '((?:api|notification)_sample_tests|functional\.db\.)' {posargs}
 
-[testenv:mypy]
+[testenv:pep8]
 description =
-  Run type checks.
-envdir = {toxworkdir}/shared
+  Run style checks.
+deps =
+  pre-commit
 extras =
 commands =
-  bash tools/mypywrap.sh {posargs}
+  pre-commit run --all-files --show-diff-on-failure
 
-[testenv:pep8]
+[testenv:mypy]
 description =
-  Run style checks.
+  Run type checks.
 deps =
   pre-commit
 extras =
 commands =
-  pre-commit run --all-files --show-diff-on-failure
+  pre-commit run --all-files --show-diff-on-failure mypy
 
 [testenv:autopep8]
 extras =