Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update build to continue to work when moved to rootless jenkins #691

Merged
merged 1 commit into from
May 6, 2022

Conversation

andytson-inviqa
Copy link
Contributor

@andytson-inviqa andytson-inviqa commented May 4, 2022

Jenkins runs docker agent nodes with --user uid:gid, so with the new runners being 1000:1000, they wont have root access in the container. Workspace image has a uid 1000 user and a 998 gid docker group set up to match the jenkins host.

Technically only the 1000 uid user is needed to solve the missing home path. The docker group could just be the gid specified in other cases

Any other images used as docker agent nodes without a 1000 uid user would either need to:

  • create that user (via a Dockerfile)
  • change HOME (or whatever other variable like COMPOSE_HOME) to either a /tmp folder or $WORKSPACE subdirectory

@andytson-inviqa andytson-inviqa force-pushed the feature/jenkins-rootless-fix branch 2 times, most recently from 0d9bb32 to 554b372 Compare May 5, 2022 16:04
Jenkins runs docker agent nodes with --user uid:gid, so with the new runners being 1000:1000, they wont have root access in the container. Workspace image has a uid 1000 user and a 998 gid docker group set up to match the jenkins host.

Technically only the 1000 uid user is needed to solve the missing home path. The docker group could just be the gid specified in other cases

Any other images used as docker agent nodes without a 1000 uid user would either need to:
* create that user
* change HOME (or whatever other variable like COMPOSE_HOME) to either a /tmp folder or $WORKSPACE subdirectory
@andytson-inviqa andytson-inviqa force-pushed the feature/jenkins-rootless-fix branch from 554b372 to 0a8766f Compare May 5, 2022 18:35
@andytson-inviqa andytson-inviqa added this to the 1.4.0 milestone May 5, 2022
@andytson-inviqa andytson-inviqa merged commit 9807449 into 1.4.x May 6, 2022
@andytson-inviqa andytson-inviqa deleted the feature/jenkins-rootless-fix branch May 6, 2022 07:58
@andytson-inviqa andytson-inviqa added enhancement New feature or request harness-all Changes required to all harness directories labels May 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request harness-all Changes required to all harness directories
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants