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/rgw: remove Federated Configuration, clean up multisite #15132

Merged
merged 4 commits into from Jun 18, 2017

Conversation

Projects
None yet
5 participants
@cbodley
Contributor

cbodley commented May 17, 2017

Region/Zone Settings
====================
Multisite Settings
=============

This comment has been minimized.

@theanalyst

theanalyst May 17, 2017

Member

maybe a ..versionadded:: 10.2.0 or so?

- ``us-east.rgw.buckets.index``
- ``us-east.rgw.buckets.data``
The zone definitions (see ``radosgw-admin zone get``) list several more

This comment has been minimized.

@theanalyst

theanalyst May 17, 2017

Member

this might also deserve a version added string? (since it is a change from Jewel - L?)

This comment has been minimized.

@theanalyst

theanalyst May 17, 2017

Member

Hm coming to think of it, this might actually make sense to add an entry in Pending/Release notes as well since it is sort of a change from previous releases to current release (ie, new deployements will have only these pools, the older deployments/upgrades will have all the pools )

This comment has been minimized.

@theanalyst

theanalyst May 17, 2017

Member

we might need to add this in the other RGW stanza which talks about pools as well

This comment has been minimized.

@cbodley

cbodley May 17, 2017

Contributor

are you referring to the Pools section in config-ref.rst that i removed? maybe this pools section should be moved into its own page, so it isn't hidden under Multisite

This comment has been minimized.

@theanalyst

theanalyst May 17, 2017

Member

yes, probably the pools section in its own page would be the best we could just use the <zone>.pool names and list them, and in case of single site deployments, these would be called default as that is the default zone name?

@@ -52,7 +52,6 @@ you may write data with one API and retrieve it with the other.
Compression <compression>
Server-Side Encryption <encryption>
Data Layout in RADOS <layout>
Upgrade to Older Versions of Jewel <upgrade_to_jewel>

This comment has been minimized.

@cbodley

cbodley May 17, 2017

Contributor

@oritwas do you agree that this page can be removed from luminous docs?

@cbodley

This comment has been minimized.

Contributor

cbodley commented May 17, 2017

@johnwilkins could you please help review the changes i made to the pools section?

@cbodley

This comment has been minimized.

Contributor

cbodley commented May 17, 2017

i pushed a copy of this branch to ceph/ceph.git, hopefully it will build a preview here:

http://docs.ceph.com/docs/wip-doc-rgw-remove-agent/radosgw/

@cbodley cbodley requested a review from yehudasa May 18, 2017

@liewegas

This comment has been minimized.

Member

liewegas commented May 19, 2017

needs rebase

@cbodley

This comment has been minimized.

Contributor

cbodley commented May 23, 2017

@yehudasa could you approve if you're okay with the pool changes?

cbodley added some commits May 17, 2017

doc/rgw: clean up multisite config options
remove region stuff, radosgw-agent stuff, and zone/pool stuff that is
duplicated in the multisite page

moved sync log config options into multisite section, added description
for rgw_run_sync_thread

Signed-off-by: Casey Bodley <cbodley@redhat.com>
doc/rgw: remove Federated Configuration page
Fixes: http://tracker.ceph.com/issues/18082

Signed-off-by: Casey Bodley <cbodley@redhat.com>
doc/rgw: remove Upgrade to Older Versions of Jewel page
Signed-off-by: Casey Bodley <cbodley@redhat.com>
doc/rgw: update pool names, document namespaces
Fixes: http://tracker.ceph.com/issues/19504

Signed-off-by: Casey Bodley <cbodley@redhat.com>
@cbodley

This comment has been minimized.

Contributor

cbodley commented Jun 16, 2017

rebased

@theanalyst

Looks good, I'll add a pending release notes entry noting the pools have changed and linking this once the docs are built

@oritwas oritwas merged commit 33a7fc0 into ceph:master Jun 18, 2017

4 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
arm64 make check arm64 make check succeeded
Details
make check make check succeeded
Details

@cbodley cbodley deleted the cbodley:wip-doc-rgw-remove-agent branch Jun 18, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment