absurdfarce commented on code in PR #1271:
URL:
https://github.com/apache/cassandra-python-driver/pull/1271#discussion_r2830489778
##########
Jenkinsfile:
##########
@@ -165,15 +165,18 @@ def getMatrixBuilds(buildContext) {
def initializeEnvironment() {
sh label: 'Initialize the environment', script: '''#!/bin/bash -lex
- pyenv global ${PYTHON_VERSION}
- sudo apt-get install socat
- pip install --upgrade pip
- pip install -U setuptools
- # install a version of pyyaml<6.0 compatible with ccm-3.1.5 as of Aug 2023
+ # One of the integration tests relies on socat so let's install that here
+ sudo apt-get install -y socat moreutils
+
+ pyenv shell ${PYTHON_VERSION}
+ python -m venv jenkins-venv
+ . ./jenkins-venv/bin/activate
Review Comment:
I moved everything in to a contained venv because we were seeing weird
behaviours in which pyenv wasn't necessarily updating the local Python instance
which in turn led to packages being applied inconsistently. To avoid all of
that it seemed worthwhile to be very explicit about the idea of a container for
all necessary Python dependencies... which is pretty much a venv.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
