Skip to content

Added upgrade guide for release-5#427

Merged
prashanthpai merged 1 commit intogluster:masterfrom
ShyamsundarR:rel-5-upgrade-guide
Oct 12, 2018
Merged

Added upgrade guide for release-5#427
prashanthpai merged 1 commit intogluster:masterfrom
ShyamsundarR:rel-5-upgrade-guide

Conversation

@ShyamsundarR
Copy link
Contributor

Attempted to not copy the guide as is, like previous iterations
and instead pointed to the 4.1 guide, as procedure remains the
same.

Signed-off-by: ShyamsundarR srangana@redhat.com

Attempted to not copy the guide as is, like previous iterations
and instead pointed to the 4.1 guide, as procedure remains the
same.

Signed-off-by: ShyamsundarR <srangana@redhat.com>
Copy link
Contributor

@prashanthpai prashanthpai left a comment

Choose a reason for hiding this comment

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

This is okay. Just want confirmation about op version bump that users have to do. The 4.1 upgrade guide page points to a generic page on op version. Is that sufficient ?

@ShyamsundarR
Copy link
Contributor Author

This is okay. Just want confirmation about op version bump that users have to do. The 4.1 upgrade guide page points to a generic page on op version. Is that sufficient ?

I read that page, and the instructions are generic enough (check and set to highest etc.) and will work as needed.

I did think of changing examples in there to at least 4.1, but other than numbers nothing changes.

Based on the above decided not to touch the same.

Copy link
Member

@joejulian joejulian left a comment

Choose a reason for hiding this comment

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

LGTM.

@prashanthpai prashanthpai merged commit 0a37bfa into gluster:master Oct 12, 2018
@ShyamsundarR ShyamsundarR deleted the rel-5-upgrade-guide branch October 18, 2018 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants