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-disk: use correct user in check_journal_req #12947

Merged
merged 1 commit into from Jan 19, 2017

Conversation

Projects
None yet
3 participants
@smatzek
Contributor

smatzek commented Jan 16, 2017

The ceph-disk tool calls ceph-osd to check the journal requirements
using OSD id 0. This creates a log file for osd-0 on the system
using the current user/group for file permissions. When ceph-disk
is run as root this makes the file owned by root which makes
the osd daemon for osd.0 unable to write to its own log file.

This commit changes the journal reqs calls of ceph-osd to pass
the ceph user and group so ceph-osd creates the log file with the
appropriate permissions.

Fixes: http://tracker.ceph.com/issues/18538

Signed-off-by: Samuel Matzek smatzek@us.ibm.com

Ceph-disk to use correct user in check_journal_req
The ceph-disk tool calls ceph-osd to check the journal requirements
using OSD id 0. This creates a log file for osd-0 on the system
using the current user/group for file permissions.  When ceph-disk
is run as root this makes the file owned by root which makes
the osd daemon for osd.0 unable to write to its own log file.

This commit changes the journal reqs calls of ceph-osd to pass
the ceph user and group so ceph-osd creates the log file with the
appropriate permissions.

Fixes: http://tracker.ceph.com/issues/18538

Signed-off-by: Samuel Matzek <smatzek@us.ibm.com>

@liewegas liewegas changed the title from Ceph-disk to use correct user in check_journal_req to ceph-disk: use correct user in check_journal_req Jan 17, 2017

@alfredodeza alfredodeza merged commit 3e75c6b into ceph:master Jan 19, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment