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: release notes for 10.2.4 #12053

Merged
merged 2 commits into from Dec 5, 2016
Merged

Conversation

theanalyst
Copy link
Member

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


This is a signal for the admin to do "ceph osd set
require_jewel_osds" - by doing this, the admin acknowledges that
there is no downgrade path.
Copy link
Member

Choose a reason for hiding this comment

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

that the upgrade is complete and no pre-jewel OSDs may be added to the cluster.

I don't want to imply that there otherwise would be a downgrade path.. there isn't (a tested) one.

Copy link
Member Author

Choose a reason for hiding this comment

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

ack

Upgrading from hammer
---------------------

Set require_jewel_osds osdmap flag on upgrade from hammer
Copy link
Member

Choose a reason for hiding this comment

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

I think you can drop this line

Copy link
Member Author

Choose a reason for hiding this comment

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

will do

Copy link
Member

@liewegas liewegas left a comment

Choose a reason for hiding this comment

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

The rest looks good!

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

ghost commented Nov 17, 2016

please test this (eio failure)

@ghost
Copy link

ghost commented Nov 17, 2016

please test this

@yuriw
Copy link
Contributor

yuriw commented Nov 17, 2016

test this please

@ghost
Copy link

ghost commented Nov 17, 2016

test this please (eio)

@ghost
Copy link

ghost commented Nov 18, 2016

jenkins test this please (eio now ignored on master)

@theanalyst
Copy link
Member Author

test this please

Copy link
Member Author

@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.

Need to add the last two prs we'll be
merging now as well

Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
@theanalyst
Copy link
Member Author

@dachary @liewegas added one more commit adding the 5 prs that made it in 10.2.4 jewel branch

@ghost
Copy link

ghost commented Dec 5, 2016

LGTM

@liewegas liewegas merged commit dded3de into ceph:master Dec 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants