-
Notifications
You must be signed in to change notification settings - Fork 13.9k
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
KAFKA-9575: Mention ZooKeeper 3.5.7 upgrade #8139
Conversation
@omkreddy @rajinisivaram Any possibility of reviewing this? A 2.5 RC is going to be cut soon, and this should be in the final release. It is just a small doc change. |
docs/upgrade.html
Outdated
@@ -42,6 +42,11 @@ <h5><a id="upgrade_250_notable" href="#upgrade_250_notable">Notable changes in 2 | |||
enabled by default. You can continue to use TLSv1 and TLSv1.1 by explicitly enabling these in the configuration options | |||
<code>ssl.protocol</code> and <code>ssl.enabled.protocols</code>. | |||
</li> | |||
<li>ZooKeeper has been upgraded to version 3.5.7. This resolves bugs that could have impacted ZooKeeper upgrades as mentioned | |||
in the 2.4.0 upgrade notes, so those comments do not apply to this version of Kafka. ZooKeeper version 3.5.7 supports TLS-encrypted connectivity |
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.
"so the workaround mentioned in 2.4 do not apply to this version of Kafka" ?
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.
@rondagostino since ZOOKEEPER-3056 is still required for any 3.4 -> 3.5.6 and above upgrade, Can we repeat upgrade description like:
ZooKeeper has been upgraded to 3.5.7. ZooKeeper upgrade from 3.4.X to 3.5.6 and above can fail if there are no snapshot files in 3.4 data directory. This usually happens in test upgrades where ZooKeeper 3.5.7 is trying to load an existing 3.4 data dir in which no snapshot file has been created. For more details about the issue please refer to ZOOKEEPER-3056. A fix is given in ZOOKEEPER-3056, which is to set snapshot.trust.empty=true config in zookeeper.properties before the upgrade.
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.
Thanks, @omkreddy. I'm assuming the part about data loss potentially occurring with snapshot.trust.empty=true
does not need to be here since https://issues.apache.org/jira/browse/ZOOKEEPER-3644 is fixed in 3.5.7. I'm also assuming we do not need to mention it in the release notes.
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.
@rondagostino Thanks for the update. LGTM.
*More detailed description of your change, if necessary. The PR title and PR message become the squashed commit message, so use a separate comment to ping reviewers.* *Summary of testing strategy (including rationale) for the feature or bug fix. Unit and/or integration tests are expected for any behaviour change and system tests should be considered for larger changes.* Author: Ron Dagostino <rdagostino@confluent.io> Reviewers: Reviewers: Manikumar Reddy <manikumar.reddy@gmail.com> Closes #8139 from rondagostino/KAFKA-9575 (cherry picked from commit d9b8b86) Signed-off-by: Manikumar Reddy <manikumar@confluent.io>
More detailed description of your change,
if necessary. The PR title and PR message become
the squashed commit message, so use a separate
comment to ping reviewers.
Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.
Committer Checklist (excluded from commit message)