Page MenuHomePhabricator

homer's tests on master don't pass on old stretch CI or on new buster CI image
Closed, ResolvedPublic

Description

In attempting to migrate homer's CI tests to run on our buster image, which includes Python 3.8, I've found that it's taking an awfully long time and eventually times out on the test installdeps steps; possibly the same problem as T270795: cumin's test dependencies trigger excessive pip backtracking? https://integration.wikimedia.org/ci/job/homer-tox-docker/179/console

Note however that the test also seem to not pass on the old stretch image we're moving from, erroring in the "py37-bandit" step: https://integration.wikimedia.org/ci/job/homer-tox-docker/178/console

Event Timeline

Change 654186 had a related patch set uploaded (by Volans; owner: Volans):
[operations/software/homer@master] tox: Remove '--skip B322' from Bandit config.

https://gerrit.wikimedia.org/r/654186

The failures on the old CI are fixed by the above patch, a newer version of bandit is failing instead of ignoring a no longer used error code to exclude.
I'm looking at the new pip resolver failure that makes pip fail with a stacktrace. It's not the same as the one for cumin unfortunately. I've also submitted the form upstream to let them aware of the failure.

Change 654186 merged by jenkins-bot:
[operations/software/homer@master] tox: Remove '--skip B322' from Bandit config.

https://gerrit.wikimedia.org/r/654186

Change 654320 had a related patch set uploaded (by Volans; owner: Volans):
[operations/software/homer@master] dependencies: remove upper limit for prospector

https://gerrit.wikimedia.org/r/654320

@Jdforrester-WMF The merged patch above should have fixed this backtracking too. Feel free to resolve this task if it works with the new pip in CI too. Thanks for letting us know.

Change 654320 merged by jenkins-bot:
[operations/software/homer@master] dependencies: remove upper limit for prospector

https://gerrit.wikimedia.org/r/654320