NO-ISSUE: Ensure data dir is writable by UID/GID

[adriengentil]

Currently, the image service fails to start using podman with a
permission error:

{“file”:“/go/src/github.com/openshift/origin/main.go:132",“func”:“main.main.func1",“level”:“fatal”,“msg”:“Failed to populate image store: failed to download https://mirror.openshift.com/pub/openshift-v4/x86_64/dependencies/rhcos/4.14/4.14.15/rhcos-4.14.15-x86_64-live.x86_64.iso: unable to create a temp file for /data/rhcos-full-iso-4.14-414.92.202402130420-0-x86_64.iso: open /data/.rhcos-full-iso-4.14-414.92.202402130420-0-x86_64.iso271460763: permission denied\n”,“time”:“2024-07-02T15:46:45Z”}

This change ensures that /data directory is writable for the UID/GID
that will be used to execute the image service executable.

[openshift-ci-robot]

@adriengentil: This pull request explicitly references no jira issue.

Details

In response to this:

Currently, the image service fails to start using podman with a
permission error:

{“file”:“/go/src/github.com/openshift/origin/main.go:132",“func”:“main.main.func1",“level”:“fatal”,“msg”:“Failed to populate image store: failed to download https://mirror.openshift.com/pub/openshift-v4/x86_64/dependencies/rhcos/4.14/4.14.15/rhcos-4.14.15-x86_64-live.x86_64.iso: unable to create a temp file for /data/rhcos-full-iso-4.14-414.92.202402130420-0-x86_64.iso: open /data/.rhcos-full-iso-4.14-414.92.202402130420-0-x86_64.iso271460763: permission denied\n”,“time”:“2024-07-02T15:46:45Z”}

This change ensures that /data directory is created with the UID/GID
that will be used to execute the image service executable.

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 openshift-eng/jira-lifecycle-plugin repository.

[adriengentil]

/retitle NO-ISSUE: Create data dir after changing UID/GID

[adriengentil]

/retitle NO-ISSUE: Ensure data dir is writable by UID/GID

[pastequo]

/lgtm

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: adriengentil, pastequo

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:

• OWNERS [adriengentil,pastequo]

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

[openshift-ci-robot]

/retest-required

Remaining retests: 0 against base HEAD f8c1e51 and 2 for PR HEAD 1054fdc in total

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 65.51%. Comparing base (9f5422b) to head (1054fdc).
Report is 3 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #243   +/-   ##
=======================================
  Coverage   65.51%   65.51%           
=======================================
  Files          24       24           
  Lines        1308     1308           
=======================================
  Hits          857      857           
  Misses        310      310           
  Partials      141      141           

[adriengentil]

/retest

[openshift-ci]

@adriengentil: 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.