Hmm I think our main is broken I m on latest main and `MODE= Pants #development

Hmm, I think our main is broken. I'm on latest mai...

gorgeous-winter-99296

01/08/2024, 10:27 PM

Hmm, I think our main is broken. I'm on latest main, and

MODE=debug pants test src/python/pants/backend/python/lint/bandit/::

fails on a clean repo. Not sure how CI is clean, unless there's some bad caching going on on my end or on GHA. Can anyone else reproduce? The error I get is our tests checking for a B303 error for using hashlib functions, whereas the actual error in the tool output is B324. I'm not sure why that'd happen unless the in-repo tool versions are unpinned somehow?

👀 1

✅ 1

better-van-82973

01/09/2024, 2:06 PM

Hmm, the test worked fine for me - I tried with both the

pants

executable from

scie-pants

and also the

./pants

wrapper script inside the repo

better-van-82973

01/09/2024, 2:10 PM

Off a fresh checkout of the repo:

Copy code

09:09:47.03 [INFO] Completed: Building pytest_runner.pex
09:10:17.15 [INFO] Completed: Run Pytest - src/python/pants/backend/python/lint/bandit/rules_integration_test.py:tests - succeeded.

✓ src/python/pants/backend/python/lint/bandit/rules_integration_test.py:tests succeeded in 30.08s.

gorgeous-winter-99296

01/09/2024, 2:11 PM

Very odd, thanks for validating. Will try nuking caches etc later.

gorgeous-winter-99296

01/09/2024, 9:19 PM

Nuking ~/.pex, ~/.cache/nce and ~/.cache/pants together did the deal.

gorgeous-winter-99296

01/09/2024, 9:58 PM

It reappears when I switch branches, despite that code not being at all touched. Will have to dig more... Might be some bandit issue maybe?

4 Views

Open in Slack

Previous Next