]> git.apps.os.sepia.ceph.com Git - ceph-ci.git/commitdiff
cephadm: fix autopep8 arg parsing on newer tox versions
authorAdam King <adking@redhat.com>
Mon, 13 Feb 2023 03:18:25 +0000 (22:18 -0500)
committerAdam King <adking@redhat.com>
Mon, 13 Feb 2023 03:18:25 +0000 (22:18 -0500)
This was failing for me on new tox versions with

fix: commands[1]> autopep8 --max-line-length 100
incorrect number of argumentsfix: exit 99 (0.06 seconds) /home/adking/ceph/src/cephadm> autopep8 --max-line-length 100 pid=309039
fix: FAIL ✖ in 0.08 seconds

As it was considering our list of args for autopep8 to each be
one separate command. Fixed by just adding slashes to the ends
of the lines (as has already been done in the tox.ini in src/pybind/mgr
for some of the tests there)

Signed-off-by: Adam King <adking@redhat.com>
src/cephadm/tox.ini

index 74b8ae909e3f236bca4951ee1b5ed03fa3b6c9ec..2cbfca70f0ed43c68c6b27d0eff6576310fddd1c 100644 (file)
@@ -10,24 +10,24 @@ skipsdist = true
 max-line-length = 100
 inline-quotes = '
 ignore =
-    E501,
+    E501, \
     W503,
 exclude =
-    .tox,
-    .vagrant,
-    __pycache__,
-    *.pyc,
-    templates,
+    .tox, \
+    .vagrant, \
+    __pycache__, \
+    *.pyc, \
+    templates, \
     .eggs
 statistics = True
 
 [autopep8]
 addopts =
-    --max-line-length {[flake8]max-line-length}
-    --ignore "{[flake8]ignore}"
-    --exclude "{[flake8]exclude}"
-    --in-place
-    --recursive
+    --max-line-length {[flake8]max-line-length} \
+    --ignore "{[flake8]ignore}" \
+    --exclude "{[flake8]exclude}" \
+    --in-place \
+    --recursive \
     --ignore-local-config
 
 [testenv]