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

rgw: move keystone secrets from ceph.conf to files #24816

Merged
merged 1 commit into from
Nov 8, 2018

Conversation

mattbenjamin
Copy link
Contributor

Permits setting restrictive permissions on these secrets.

Fixes: http://tracker.ceph.com/issues/36621

Signed-off-by: Matt Benjamin mbenjamin@redhat.com

  • References tracker ticket
  • Updates documentation if necessary
  • Includes tests for new functionality or reproducer for bug

@mattbenjamin mattbenjamin force-pushed the wip-rgw-keystone-secret branch 2 times, most recently from 335da1f to 523cfb9 Compare November 2, 2018 00:29
@mattbenjamin mattbenjamin self-assigned this Nov 2, 2018
@mattbenjamin
Copy link
Contributor Author

@cbodley read_secret tested w/input of 9 and 2K

@cbodley
Copy link
Contributor

cbodley commented Nov 2, 2018

code looks good. i like the deprecation strategy. could you please update http://docs.ceph.com/docs/master/radosgw/keystone/ to reflect the changes?

@mattbenjamin
Copy link
Contributor Author

@cbodley added doc

@mattbenjamin mattbenjamin force-pushed the wip-rgw-keystone-secret branch 2 times, most recently from 348325b to 0ab7cff Compare November 2, 2018 13:57
doc/radosgw/config-ref.rst Outdated Show resolved Hide resolved
Copy link
Contributor

@cbodley cbodley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks great otherwise

Permits setting restrictive permissions on these secrets.

Fixes: http://tracker.ceph.com/issues/36621

Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
@cbodley
Copy link
Contributor

cbodley commented Nov 8, 2018

@cbodley cbodley merged commit b97e471 into ceph:master Nov 8, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants