]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
pybind/mgr/tox.ini: add commas in "modules" variable 49490/head
authorKefu Chai <tchaikov@gmail.com>
Sun, 18 Dec 2022 12:18:44 +0000 (20:18 +0800)
committerKefu Chai <tchaikov@gmail.com>
Sun, 18 Dec 2022 12:23:47 +0000 (20:23 +0800)
since tox v4.0.13, it parses the variables differently, so the newlines
in a variable are passed right to the command referencing it. so we now
have failure like:
```
flake8: commands[0] /home/jenkins-build/build/workspace/ceph-pull-requests/src/pybind/mgr> flake8 --config=tox.ini alerts
flake8: commands[1] /home/jenkins-build/build/workspace/ceph-pull-requests/src/pybind/mgr> balancer
flake8: exit 2 (0.00 seconds) /home/jenkins-build/build/workspace/ceph-pull-requests/src/pybind/mgr> balancer
flake8: FAIL ✖ in 3.33 seconds
```
so we have to add comma as line continuation separator to address
this problem.

Signed-off-by: Kefu Chai <tchaikov@gmail.com>
src/pybind/mgr/tox.ini

index 85e7ae3db11bb84cbb6f892e14199ab98873d43d..aaabd7ba6404b698a37a406a08d9337e2b256e49 100644 (file)
@@ -158,20 +158,20 @@ deps =
     flake8
 allowlist_externals = bash
 modules =
-    alerts
-    balancer
-    cephadm
-    cli_api
-    crash
-    devicehealth
-    diskprediction_local
-    hello
-    iostat
-    localpool
-    nfs
-    orchestrator
-    prometheus
-    rbd_support
+    alerts \
+    balancer \
+    cephadm \
+    cli_api \
+    crash \
+    devicehealth \
+    diskprediction_local \
+    hello \
+    iostat \
+    localpool \
+    nfs \
+    orchestrator \
+    prometheus \
+    rbd_support \
     selftest
 commands =
     flake8 --config=tox.ini {posargs} \