OCPBUGS-89330: Fix cache path to avoid /var/cache/dnf conflict - #145
OCPBUGS-89330: Fix cache path to avoid /var/cache/dnf conflict#145ericahinkleRH wants to merge 1 commit into
Conversation
|
@ericahinkleRH: This pull request references Jira Issue OCPBUGS-89330, which is invalid:
Comment The bug has been updated to refer to the pull request using the external bug tracker. DetailsIn response to this:
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. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: ericahinkleRH The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
…/var/cache/dnf conflict The default cache directory was /var/cache/, which caused chmod failures when running as non-root (UID 1001) because the RHEL base image contains /var/cache/dnf owned by root. EnsureEmptyDirectory() attempts to chmod entries in the cache directory to make them writable before deletion, but non-root users cannot chmod root-owned directories, resulting in: 'chmod /var/cache/dnf: operation not permitted' Fix: Change default from /var/cache/ to /var/cache/catalogd to avoid conflicts with system directories in the base image.
a4c66f6 to
4dfd672
Compare
|
/test all |
|
@ericahinkleRH: No presubmit jobs available for openshift/operator-framework-catalogd@main DetailsIn response to this:
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. |
|
/jira refresh |
|
@ericahinkleRH: This pull request references Jira Issue OCPBUGS-89330, which is valid. The bug has been moved to the POST state. 3 validation(s) were run on this bug
DetailsIn response to this:
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. |
|
Closing this PR - catalogd functionality was merged into operator-controller upstream, so the fix only needs to go there. See upstream PR: operator-framework/operator-controller#2835 |
|
@ericahinkleRH: This pull request references Jira Issue OCPBUGS-89330. The bug has been updated to no longer refer to the pull request using the external bug tracker. All external bug links have been closed. The bug has been moved to the NEW state. DetailsIn response to this:
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. |
Problem
On OpenShift 4.19.32, catalogd pods crash on startup with:
This happens because:
/var/cache//var/cache/dnfowned by rootEnsureEmptyDirectory()tries to chmod root-owned directories, which failsSolution
Change default cache directory from
/var/cache/to/var/cache/catalogdto avoid conflicts with system directories.Testing
Fixes: https://redhat.atlassian.net/browse/OCPBUGS-89330