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

hammer: keyring permisions for mon deamon #8049

Merged
1 commit merged into from
Apr 6, 2016

Conversation

xiaoxichen
Copy link
Contributor

The command:

sudo ceph-mon --cluster ceph --mkfs -i $NODE_NAME --keyring $PATH_MON_KEYRING

Writes a new keyring with permissions set to "0644".

Fixes: ceph#14950
Signed-off-by: Owen Synge osynge@suse.com
(cherry picked from commit c2f91a8)
@xiaoxichen xiaoxichen self-assigned this Mar 11, 2016
@xiaoxichen xiaoxichen added this to the hammer milestone Mar 11, 2016
ghost pushed a commit that referenced this pull request Apr 5, 2016
Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost
Copy link

ghost commented Apr 6, 2016

It passed the hammer upgrade suite ( http://tracker.ceph.com/issues/14692#note-52 ) and the rados suite http://tracker.ceph.com/issues/14692#note-51).

@ghost ghost merged commit d08bae2 into ceph:hammer Apr 6, 2016
@xiaoxichen xiaoxichen deleted the wip-15022-hammer branch April 7, 2016 02:57
This pull request was closed.
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.

2 participants