Skip to content

Commit

Permalink
Shift creation and activation of venv back to Dockerfile
Browse files Browse the repository at this point in the history
Making a venv at /venv requires root permissions as well, as it turns
out. Using it will be as well so we chown it.
  • Loading branch information
james-smith-za committed Dec 3, 2024
1 parent f97db81 commit 2bf3f2e
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
4 changes: 4 additions & 0 deletions .ci/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -27,3 +27,7 @@ COPY install-sys-pkgs.sh .
# jq and iproute2 are used to find the interface IPv4 address
RUN apt-get update && apt-get -y --no-install-recommends install libpython3-dev python3-venv jq iproute2
RUN ./install-sys-pkgs.sh

RUN python3 -m venv /venv
RUN chown -R +1000:+1000 /venv
ENV PATH=/venv/bin:$PATH
3 changes: 1 addition & 2 deletions Jenkinsfile
Original file line number Diff line number Diff line change
Expand Up @@ -44,8 +44,7 @@ pipeline {
stages {
stage('Install dependencies') {
steps {
sh 'python3 -m venv /venv'
sh 'PATH="/venv/bin:$PATH" py-requirements.sh'
sh '.ci/py-requirements.sh'

}
}
Expand Down

0 comments on commit 2bf3f2e

Please sign in to comment.