Adjust openstack_keys

[noonedeadpunk]

Since glance do not need write access to volumes pool and
set permission is static (do not rely on openstack_cinder_pool)
we can easily drop it.

Also as glance do not need write access to volumes pool and
set permission is static (do not rely on openstack_cinder_pool)
we can easily drop it.

[noonedeadpunk]

retest this please

[noonedeadpunk]

retest this please

[noonedeadpunk]

retest this please

[guits]

@gfidente @fultonj @fmount

[gfidente]

hi, thanks for your submission

can you please amend the class-read addition?

[noonedeadpunk]

retest this please

[guits]

@logan2211 from what I read here, I understand @noonedeadpunk should set back the class-read object_prefix rbd_children permission on this patch?

[noonedeadpunk]

@logan2211 thanks for stepping in and describing the problem in details (while I was really missing them)
@guits I've returned object_prefix rbd_children

[guits]

@noonedeadpunk sounds good to me
@logan2211 thanks, I think it's pretty clear for everyone unless @gfidente has something different to share?

[noonedeadpunk]

retest this please

[gfidente]

Considering the previous implementation matched what was suggested in ceph docs at [1], maybe this should trigger a change in the docs as well?

1. https://docs.ceph.com/docs/master/rbd/rbd-openstack/

[noonedeadpunk]

Considering the previous implementation matched what was suggested in ceph docs at [1], maybe this should trigger a change in the docs as well?

1. https://docs.ceph.com/docs/master/rbd/rbd-openstack/

Placed ceph/ceph#29865 to update docs

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[gfidente]

@guits I thought we wanted this one in?

[noonedeadpunk]

Rebased on top of the master just in case

[dsavineau]

@noonedeadpunk Could you amend your commit message ? because it's mentioned cinder & glance but the code is only modifying the cinder caps.

[noonedeadpunk]

@dsavineau Good catch, as I've lost volumes permission drop during rebase.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[noonedeadpunk]

retest this please

[noonedeadpunk]

@guits @gfidente So lot's of time passed since PR submission. Considering that caph-ansible will be sunseted in favor of the new deployment system - does this patch still has any sense?

[dsavineau]

jenkins test centos-container-all_daemons

[dsavineau]

jenkins test centos-container-update

[dsavineau]

jenkins test centos-container-purge

[dsavineau]

retest this please

https://docs.ceph.com/ceph-ansible/master/dev/index.html#pull-request-testing

@guits @gfidente So lot's of time passed since PR submission. Considering that caph-ansible will be sunseted in favor of the new deployment system - does this patch still has any sense?

What new deployment system are you talking about ?

[noonedeadpunk]

@dsavineau I was talking about ceph orchestrator, which is going to replace ceph-deploy, ceph-ansible, etc in octopus
https://docs.google.com/presentation/d/1JpcETNXpuB1JEuhX_c8xtgNnv0gJhSaQUffM7aRjUek

[dsavineau]

AFAIK ceph-ansible will still be there for octopus

[guits]

AFAIK ceph-ansible will still be there for octopus

correct

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.