oms4suse / python-ceph-cfg

Simple API to help deploying ceph.
Other
7 stars 12 forks source link

keyring.py: Fix auid/set-uid settings of CEPHX #4

Closed hoonetorg closed 8 years ago

hoonetorg commented 8 years ago

When browsing in history of this git repo it can be seen that between commit 09398b4 and commit 59480e3 went setting auid=0/--set-uid=0 from client.admin key to mon key. I'm not sure if this is correct. AFAIK --set-uid=0/auid=0 was always for client admin key at least until hammer. When doing a test deployment with ceph-deploy on jewel --set-uid=0/auid=0 is not done anymore on any keyring. This may have to do with switching ceph from running as root to running as ceph user. That's why --set-uid=0 is moved from mon to client.admin and only used if version is not larger than hammer.

Signed-off-by: Holger Fischer holger.fischer@hoonet.org