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: add recovery procedure for upgrade to older version of jewel #11827

Merged
merged 1 commit into from Nov 21, 2016

Conversation

oritwas
Copy link
Member

@oritwas oritwas commented Nov 8, 2016

Fixes: http://tracker.ceph.com/issues/17820
Signed-off-by: Orit Wasserman owasserm@redhat.com

@@ -0,0 +1,37 @@
=============
RGW upgrade procedure to older version of jewel
Copy link
Member

Choose a reason for hiding this comment

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

from older versions to Jewel

Copy link
Member

Choose a reason for hiding this comment

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

ah.. now I understand.. maybe: upgrading to Jewel versions 10.2.0, 10.2.1, 10.2.2

Copy link
Member

Choose a reason for hiding this comment

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

So is this the doc. for upgrading to jewel versions upto 10.2.4? Is this applicable for 10.2.4 as well?

Copy link
Member Author

Choose a reason for hiding this comment

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

The issue was fixed in 10.2.4

Copy link
Member Author

Choose a reason for hiding this comment

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

so for upgrading to 10.2.4 you don't need the workaround


Upgrade of :term:`Ceph Object Gateway` to older versions of jewel (up to 10.2.3) caused issues. This document describes the needed recovery procedure.

Mixed version of :term:`Ceph Object Gateway` is supported only from 10.2.4
Copy link
Member

Choose a reason for hiding this comment

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

We shouldn't really say we support it, could be misunderstood.


Run the following commands:::

$ radosgw-admin zonnegroup get --rgw-zonegroup=default | sed 's/"id":.*/"id": "default",/g' | sed 's/"master_zone.*/"master_zone":"default",/g' > default-zg.json
Copy link
Member

Choose a reason for hiding this comment

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

zonegroup get

@@ -0,0 +1,37 @@
=============
RGW upgrade procedure to older version of jewel
Copy link
Member

Choose a reason for hiding this comment

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

So is this the doc. for upgrading to jewel versions upto 10.2.4? Is this applicable for 10.2.4 as well?


$ radosgw-admin zonegroup default --rgw-zonegroup=default

$ radosgw-adminzone default --rgw-zone=default
Copy link
Contributor

Choose a reason for hiding this comment

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

radosgw-admin zone


$ radosgw-admin zonegroup default --rgw-zonegroup=default

$ radosgw-admin zone default --rgw-zone=default
Copy link
Contributor

Choose a reason for hiding this comment

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

nit the zonegroup set and zone set commands will accept the --default flag, so you could avoid these last two steps

and am i correct that a radosgw-admin period update --commit isn't needed here because radosgw will fall back to the local zone/zonegroup objects?

Copy link
Member Author

Choose a reason for hiding this comment

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

correct.
A side note the period update --commit actually fails here http://tracker.ceph.com/issues/17364.
I looking at it at the moment

Copy link
Member

@theanalyst theanalyst left a comment

Choose a reason for hiding this comment

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

LGTM

@theanalyst
Copy link
Member

jenkins test this please

@cbodley cbodley merged commit 139452c into ceph:master Nov 21, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants