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

ceph-defaults: use ceph as user and group #7560

Merged
merged 3 commits into from
May 17, 2024

Conversation

clwluvw
Copy link
Collaborator

@clwluvw clwluvw commented May 16, 2024

To avoid mixup permissions on host and ceph-crash runs as ceph which can't send crash reports generated by others with root user.

@clwluvw clwluvw changed the title ceph-defaults: use ceph as user and group [skip ci] ceph-defaults: use ceph as user and group May 16, 2024
@clwluvw clwluvw changed the title [skip ci] ceph-defaults: use ceph as user and group ceph-defaults: use ceph as user and group May 16, 2024
@clwluvw
Copy link
Collaborator Author

clwluvw commented May 16, 2024

jenkins test centos-container-collocation

@clwluvw clwluvw changed the title ceph-defaults: use ceph as user and group [skip ci] ceph-defaults: use ceph as user and group May 16, 2024
@clwluvw clwluvw changed the title [skip ci] ceph-defaults: use ceph as user and group ceph-defaults: use ceph as user and group May 16, 2024
@clwluvw
Copy link
Collaborator Author

clwluvw commented May 16, 2024

jenkins test centos-container-collocation

Make it the same as "Create ceph initial directories" task
and make owner and group 167 for containers so they can write
with ceph user.

Signed-off-by: Seena Fallah <seenafallah@gmail.com>
Signed-off-by: Seena Fallah <seenafallah@gmail.com>
Use --setuser and --setgroup to ceph to run daemons with ceph user.

Signed-off-by: Seena Fallah <seenafallah@gmail.com>
@guits guits merged commit 7016c6d into ceph:main May 17, 2024
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants