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

doc: update the rados namespace docs #15838

Merged
merged 1 commit into from Aug 1, 2017

Conversation

Projects
None yet
3 participants
@theanalyst
Copy link
Member

theanalyst commented Jun 22, 2017

Since last update was as of firefly and we do use namespaces in RGW

Signed-off-by: Abhishek Lekshmanan abhishek@suse.com

@theanalyst theanalyst requested review from liewegas and cbodley Jun 22, 2017

@theanalyst

This comment has been minimized.

Copy link
Member Author

theanalyst commented Jun 22, 2017

jenkins test this please

.. note:: namespaces are primarily useful for applications written on top of
``librados`` where the logical grouping can alleviate the need to create
different pools. Ceph radosgw (from ``luminous``) uses namespaces for various
metadata objects.

This comment has been minimized.

Copy link
@cbodley

cbodley Jun 22, 2017

Contributor

capitalize Namespaces at the beginning of the sentence, and Ceph radosgw -> Ceph Object Gateway

This comment has been minimized.

Copy link
@theanalyst

theanalyst Jun 22, 2017

Author Member

ack

doc: update the rados namespace docs
Since last update was as of firefly and we use namespaces in RGW

Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>

@theanalyst theanalyst force-pushed the theanalyst:doc-pool-ns-update branch from 42a6f38 to 4a3c827 Jul 26, 2017

@theanalyst

This comment has been minimized.

Copy link
Member Author

theanalyst commented Jul 26, 2017

@cbodley updated, sorry for letting this slide

@cbodley

This comment has been minimized.

Copy link
Contributor

cbodley commented Jul 26, 2017

jenkins test this please

@theanalyst

This comment has been minimized.

Copy link
Member Author

theanalyst commented Aug 1, 2017

@cbodley Ok to merge?

@liewegas liewegas merged commit 6d7e4b4 into ceph:master Aug 1, 2017

4 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details
make check (arm64) make check succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.