From 0cc649f029d40555e1d059e3de3739be7b9ae197 Mon Sep 17 00:00:00 2001 From: Kefu Chai Date: Sun, 18 Dec 2022 20:18:44 +0800 Subject: [PATCH] pybind/mgr/tox.ini: add commas in "modules" variable MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit 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 --- src/pybind/mgr/tox.ini | 28 ++++++++++++++-------------- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/src/pybind/mgr/tox.ini b/src/pybind/mgr/tox.ini index 85e7ae3db11bb..aaabd7ba6404b 100644 --- a/src/pybind/mgr/tox.ini +++ b/src/pybind/mgr/tox.ini @@ -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} \ -- 2.39.5