Support for mgr caps in keys

Bug #1720864 reported by Dan Williams
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
puppet-ceph
Fix Released
Undecided
Dan Williams

Bug Description

Since the introduction of the mgr it seems things such as ceph pg dump require mgr caps.
the key class should be extended to all specifying the mgr caps.

They can be set to undef by default as with other key caps and then only used if the end user requires them.

Revision history for this message
Dan Williams (adventsol) wrote :

I've submitted a pull request. Please let me know if there's anything I need to change.

https://github.com/openstack/puppet-ceph/pull/32

Revision history for this message
Dan Williams (adventsol) wrote :

Just got the email through, i’ll Resubmit it via the review process.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-ceph (master)

Fix proposed to branch: master
Review: https://review.openstack.org/509262

Changed in puppet-ceph:
assignee: nobody → Daniel Williams (adventsol)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-ceph (master)

Reviewed: https://review.openstack.org/509262
Committed: https://git.openstack.org/cgit/openstack/puppet-ceph/commit/?id=5db96f877d07480b4bdf95542a3fdb0210d6baca
Submitter: Jenkins
Branch: master

commit 5db96f877d07480b4bdf95542a3fdb0210d6baca
Author: Dan Williams <email address hidden>
Date: Tue Oct 3 20:32:19 2017 +0100

    Allows mgr caps to be added to keys.

    Allows setting optional mgr caps on the cephx keys.
    The mgr caps are required to query the ceph-mgr daemon.

    Change-Id: I1c4d010b55611377573e4b928ee9527441050d51
    Closes-Bug: #1720864

Changed in puppet-ceph:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/puppet-ceph 2.5.0

This issue was fixed in the openstack/puppet-ceph 2.5.0 release.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.