Skip to content

Update Jenkinsfile to run on Mantle#96

Merged
lispandfound merged 6 commits intomasterfrom
jenkins
Mar 13, 2025
Merged

Update Jenkinsfile to run on Mantle#96
lispandfound merged 6 commits intomasterfrom
jenkins

Conversation

@lispandfound
Copy link
Contributor

Jenkins now runs jobs on Mantle, this requires a change to the Jenkinsfile to run as the root user inside the python container.

@lispandfound
Copy link
Contributor Author

We're going to have the same problem as another PR. I've already run this build with Jenkins just before: https://quakecoresoft.canterbury.ac.nz/jenkins/job/visualization/63/. Jenkins believes it has done its job because a build has already succeeded for this commit. So it never runs the build and we get stuck in infinite limbo.

Copy link
Member

@sungeunbae sungeunbae left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What does this mean?

@lispandfound
Copy link
Contributor Author

lispandfound commented Mar 3, 2025

What does this mean?

@sungeunbae the -u 0 flag means to run inside the container as the user with uid = 0. This user is, by definition, root. So it's effectively saying "run commands in this container as root".

@lispandfound
Copy link
Contributor Author

@sungeunbae I've added a comment above that line explaining the effect of -u 0.

@lispandfound lispandfound merged commit e48526c into master Mar 13, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants