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

edit /etc/security/limits.conf after installing ceph #2197

Merged
merged 1 commit into from
Mar 28, 2024
Merged

Conversation

ktdreyer
Copy link
Member

@ktdreyer ktdreyer commented Mar 28, 2024

The pam RPM (and /etc/security directory) is not present in the ubi9-minimal base images. For minimal images, pam is only available after we install ceph.

Edit the /etc/security/limits.conf file after ceph (and pam) is installed.

This fixes this error introduced in #2194:

/bin/sh: line 1: /etc/security/limits.conf: No such file or directory

The pam RPM (and /etc/security directory) is not present in the
ubi9-minimal base images. For minimal images, pam is only available
after we install ceph.

Edit the /etc/security/limits.conf file after ceph (and pam) is
installed.

This fixes this error:

  /bin/sh: line 1: /etc/security/limits.conf: No such file or directory

Signed-off-by: Ken Dreyer <[email protected]>
@tserlin
Copy link
Collaborator

tserlin commented Mar 28, 2024

This just affected us downstream in our container builds, with the same error, as the dist-git "containers/rhceph" repo was updated a few hours ago.

@guits guits merged commit bc74500 into main Mar 28, 2024
8 checks passed
@guits guits deleted the qat-ordering branch March 28, 2024 16:43
@tserlin
Copy link
Collaborator

tserlin commented Mar 28, 2024

@guits Can you sync the change with downstream? I tried a 7.1 scratch container build downstream with the change manually applied and it does build for me 👍

@ktdreyer
Copy link
Member Author

ktdreyer commented Apr 1, 2024

We'll need this in rhceph and ibm-ceph downstream.

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.

3 participants