]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
pybind/mgr: remove vestigial basepython directives 56821/head
authorJohn Mulligan <jmulligan@redhat.com>
Wed, 10 Apr 2024 14:38:29 +0000 (10:38 -0400)
committerJohn Mulligan <jmulligan@redhat.com>
Wed, 10 Apr 2024 14:38:29 +0000 (10:38 -0400)
Remove what I think are vestigial configuration parameters setting
`basepython` in tox.ini. These may have been useful during the change
from python 2 to 3, but I don't see them serving any useful purpose
at this current time. In fact they now interfere with the ability to
globally override the basepython version of all testenv's at once.
So let's just remove them. We can always add it back if we find
an issue in the future (and document it!).

Signed-off-by: John Mulligan <jmulligan@redhat.com>
src/pybind/mgr/tox.ini

index 082d0dbd53c2571b727ef16dc564a18b9ed90305..61f3643e8900c239002eb38db67079464030acc6 100644 (file)
@@ -146,7 +146,6 @@ commands =
       {posargs:{[testenv:fix]modules}}
 
 [testenv:pylint]
-basepython = python3
 deps =
     pylint
 modules =
@@ -155,7 +154,6 @@ commands =
     pylint {[pylint]addopts} {posargs:{[testenv:pylint]modules}}
 
 [testenv:flake8]
-basepython = python3
 deps =
     flake8
 allowlist_externals = bash
@@ -182,7 +180,6 @@ commands =
     bash -c 'test $(git ls-files cephadm | grep ".py$" | grep -v tests | xargs grep "docker.io" | wc -l) == 13'
 
 [testenv:jinjalint]
-basepython = python3
 deps =
     jinjaninja
 commands =