-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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] Upgrade for 0.12 #1659
[DOC] Upgrade for 0.12 #1659
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. Just one comment.
@scholzj - are we ready to merge? |
No, I haven't reviewed it yet. |
(I will do it this afternoon)
…On Fri, May 31, 2019, 11:31 PaulRMellor ***@***.***> wrote:
@scholzj <https://github.com/scholzj> - are we ready to merge?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1659?email_source=notifications&email_token=ABLFOR47PKSDSZFLVOVX4PLPYDV5TA5CNFSM4HQLRPEKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWUXCUI#issuecomment-497643857>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABLFOR5N4WV3N4AEH3ESNGTPYDV5TANCNFSM4HQLRPEA>
.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks quite good. My main concern is that this does not make it clear that it talks about the upgrade from 0.11.x to 0.12.x. We should make it more clear that for any older version upgrades the users should look up the older docs.
Type of change
Description
Updates to the Upgrade chapter
Following consultation, the x to y Cluster Operator upgrade procedures have been replaced with a generic cluster operator upgrade procedure. The generic procedure now includes a link to the resource updates chapter in the final step.
Additionally, the attributes used in the Kafka upgrade procedures -- lower and higher Kafka version and log version -- have been updated to 2.1.1 and 2.2.0 and 2.1 and 2.2 respectively.
Checklist
Please go through this checklist and make sure all applicable tasks have been done
./design