Skip to content
This repository was archived by the owner on Aug 13, 2024. It is now read-only.

Conversation

@jakubgs
Copy link
Member

@jakubgs jakubgs commented Apr 15, 2024

Desktop QA tests also use Pytest packages and their versions are different, so we can't install them globally, it needs to be done per build using WORKSPACE_TMP as destination.

Otherwise you get this due to a race condition:

 > a ci-slave-linux,ci-slave-release -o -a 'sudo -u jenkins pip list --user 2>/dev/null | grep "^pytest "' | sort
linux-01.he-eu-hel1.ci.release | CHANGED | rc=0 | (stdout) pytest  7.4.0
linux-02.he-eu-hel1.ci.release | CHANGED | rc=0 | (stdout) pytest  7.4.0
linux-01.he-eu-hel1.ci.devel | CHANGED | rc=0 | (stdout) pytest    7.2.1
linux-02.he-eu-hel1.ci.devel | CHANGED | rc=0 | (stdout) pytest    7.4.0
linux-03.he-eu-hel1.ci.devel | CHANGED | rc=0 | (stdout) pytest    7.4.0
linux-04.he-eu-hel1.ci.devel | CHANGED | rc=0 | (stdout) pytest    7.4.0
linux-05.he-eu-hel1.ci.devel | CHANGED | rc=0 | (stdout) pytest    7.2.1

Related to:

@jakubgs jakubgs added the CI Jenkins' related stuff label Apr 15, 2024
@jakubgs jakubgs self-assigned this Apr 15, 2024
Desktop QA tests also use Pytest packages and their versions are
different, so we can't install them globally, it needs to be done per
build using `WORKSPACE_TMP` as destination.

Signed-off-by: Jakub Sokołowski <jakub@status.im>
@jakubgs jakubgs force-pushed the ci/venv-for-python-deps branch from a5d6a86 to b65bdbd Compare April 15, 2024 14:12
@jakubgs jakubgs requested a review from anastasiyaig April 15, 2024 14:44
@anastasiyaig anastasiyaig merged commit 14caa9e into master Apr 15, 2024
@anastasiyaig anastasiyaig deleted the ci/venv-for-python-deps branch April 15, 2024 14:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

CI Jenkins' related stuff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants