some tiny fixes of ceph

blueboxgroup / ursula

Ansible playbooks for operating OpenStack - Powering Blue Box Cloud.

https://www.blueboxcloud.com

204 stars 5 forks source link

some tiny fixes of ceph #2848

Closed chengtcli closed 7 years ago

chengtcli commented 7 years ago

Remove the task which changes owner of admi keyring file. It was used for upgrading from hammer to Jewel.
Add run_once for task which changes cap of CEPH users.
Fix default('True')|bool

bbc-jenkins commented 7 years ago

Can one of the admins verify this patch?

\"ok to test\" to accept this pull request for testing
\"test this please\" for a one time test run
\"add to whitelist\" to add the author to the whitelist

bbc-jenkins commented 7 years ago

Can one of the admins verify this patch?

\"ok to test\" to accept this pull request for testing
\"test this please\" for a one time test run
\"add to whitelist\" to add the author to the whitelist

bbc-jenkins commented 7 years ago

Can one of the admins verify this patch?

\"ok to test\" to accept this pull request for testing
\"test this please\" for a one time test run
\"add to whitelist\" to add the author to the whitelist

chengtcli commented 7 years ago

tested on tardis VMs env: envs/example/ci-ceph-ubuntu

After deployment, CEPH admin keyring is owned by ceph user as expected


root@cheng-cpm1:~# ll /etc/ceph
total 40
drwxr-xr-x   2 root root 4096 May 18 11:14 ./
drwxr-xr-x 104 root root 4096 May 18 10:37 ../
-rw-------   1 ceph ceph   63 May 18 10:37 ceph.client.admin.keyring```

nirajdp76 commented 7 years ago

ok to test