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 some RGW documentation #15175

Merged
merged 1 commit into from May 24, 2017

Conversation

Projects
None yet
3 participants
@jharbott
Contributor

jharbott commented May 19, 2017

  • Update RGW quickstart doc to match current status
  • Use --rgw_num option with vstart.sh

Signed-off-by: Jens Rosenboom j.rosenboom@x-ion.de

@theanalyst theanalyst requested a review from cbodley May 19, 2017

.. code::
$ cd build
$ ../src/vstart.sh -d -n -x -r
$ ../src/vstart.sh -d -n -x --num_rgw 1

This comment has been minimized.

@liewegas

liewegas May 19, 2017

Member

We should prefix with RGW=1 instead of using --num_rgw (which I suggest we remove!)

This comment has been minimized.

@jharbott

jharbott May 22, 2017

Contributor

Done

@@ -178,22 +183,22 @@ Known pools:
prodtx$prodt.buckets
test2
.users.email
namespace: users.email

This comment has been minimized.

@cbodley

cbodley May 19, 2017

Contributor

the formatting doesn't turn out very well here, this is how it renders:

namespace: users.email Unimportant

maybe try adding another level of indentation under the namespaces? ie

<zone>.rgw.meta
  ...
  namespace: users.email
    Unimportant

This comment has been minimized.

@jharbott

jharbott May 22, 2017

Contributor

New version pushed, also fixing some pre-existing formatting issues. There's still room for further improvement, but I think this should work for now.

Unimportant
.users
namespace: users.keys

This comment has been minimized.

@cbodley

cbodley May 19, 2017

Contributor

while you're here, could you please replace this text:
It's unclear why user ID is not used to name objects in this pool.
with this:
This allows radosgw to look up users by their access keys during authentication.

This comment has been minimized.

@jharbott

jharbott May 22, 2017

Contributor

Done

Jens Rosenboom
Update some RGW documentation
- Update RGW quickstart doc to match current status
- Use --rgw_num option with vstart.sh

Signed-off-by: Jens Rosenboom <j.rosenboom@x-ion.de>
@cbodley

formatting looks way better 👍

@cbodley

This comment has been minimized.

Contributor

cbodley commented May 24, 2017

ping @liewegas

@liewegas liewegas changed the title from Update some RGW documentation to doc: Update some RGW documentation May 24, 2017

@liewegas liewegas merged commit 773a3f3 into ceph:master May 24, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment