Skip to content

fix: drop USER 1001 from Dockerfile.builder for ci-operator - #84

Merged
openshift-merge-bot[bot] merged 1 commit into
openshift:masterfrom
pmeida:fix/dockerfile-builder-ci
Aug 4, 2026
Merged

fix: drop USER 1001 from Dockerfile.builder for ci-operator#84
openshift-merge-bot[bot] merged 1 commit into
openshift:masterfrom
pmeida:fix/dockerfile-builder-ci

Conversation

@pmeida

@pmeida pmeida commented Aug 4, 2026

Copy link
Copy Markdown

Summary

  • Remove USER 1001 from Dockerfile.builder so ci-operator's clonerefs can write to /go/src/ when using this image as build_root project_image
  • This is a CI tool image, not a production one, so running as root is fine - I saw a similar implementation that does the same

Problem

https://prow.ci.openshift.org/view/gs/test-platform-results/pr-logs/pull/openshift_release/82896/rehearse-82896-pull-ci-openshift-faas-console-plugin-master-e2e-aws/2084651189235879936
The src build step fails with:

mkdir: cannot create directory '/go': Permission denied

clonerefs tries to clone sources into /go/src/... but USER 1001 prevents writing to /go.

@openshift-ci
openshift-ci Bot requested review from Cragsmann and twoGiants August 4, 2026 15:42
@matejvasek
matejvasek requested a review from dsimansk August 4, 2026 15:47
ci-operator's clonerefs needs to write to /go/src/ when using this
image as build_root project_image. USER 1001 causes 'Permission
denied' on /go. Since this is a CI tool image (not production),
running as root is fine.

Signed-off-by: Pedro Almeida <pealmeid@redhat.com>
@openshift-ci

openshift-ci Bot commented Aug 4, 2026

Copy link
Copy Markdown

@pmeida: all tests passed!

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@matejvasek

Copy link
Copy Markdown

/approve
/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Aug 4, 2026
@openshift-ci

openshift-ci Bot commented Aug 4, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: matejvasek, pmeida

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 4, 2026
@openshift-merge-bot
openshift-merge-bot Bot merged commit b6393c7 into openshift:master Aug 4, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants