On Wed, Apr 22, 2020 at 8:48 PM Nir Soffer <nsoffer@redhat.com> wrote:
Looks like something is installing stuff in .local/ inside the project source directory, and flake8 fails since we run it like this:
$ python /home/jenkins/workspace/vdsm_standard-check-patch/vdsm/tests/profile flake8 flake8 --statistics . build-aux/vercmp contrib/logdb contrib/logstat contrib/lvs-stats contrib/profile-stats init/daemonAdapter lib/vdsm/storage/curl-img-wrap lib/vdsm/storage/fc-scan static/libexec/vdsm/get-conf-item static/usr/bin/vdsm-tool
And it tries to check files in .local/:
[2020-04-22T13:55:27.086Z] /home/jenkins/workspace/vdsm_standard-check-patch/vdsm/.tox/flake8/lib/python3.7/site-packages/pycodestyle.py:113: FutureWarning: Possible nested set at position 1 [2020-04-22T13:55:27.086Z] EXTRANEOUS_WHITESPACE_REGEX = re.compile(r'[[({] | []}),;:]') [2020-04-22T13:57:04.906Z] ./.local/share/virtualenv/seed-app-data/v1.0.1/3.7/image/CopyPipInstall/setuptools-46.1.3-py3-none-any/pkg_resources/_vendor/appdirs.py:13:80: E501 line too long (96 > 79 characters) [2020-04-22T13:57:04.906Z] # - Mac OS X: http://developer.apple.com/documentation/MacOSX/Conceptual/BPFileSystem/inde... [2020-04-22T13:57:04.906Z] ^ [2020-04-22T13:57:04.906Z] ./.local/share/virtualenv/seed-app-data/v1.0.1/3.7/image/CopyPipInstall/setuptools-46.1.3-py3-none-any/pkg_resources/_vendor/appdirs.py:14:80: E501 line too long (93 > 79 characters)
I remember that we already handled this issue, not sure why it pops again.
Found the issue, someone pushed a patch rebased on an ancient version before we fixed this issue.
We can solve this by more strict search path instead of .
Nir