-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Bump kafka and zookeeper versions #2988
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found Additional details and impacted files@@ Coverage Diff @@
## master #2988 +/- ##
=======================================
Coverage 99.01% 99.01%
=======================================
Files 3 3
Lines 203 203
=======================================
Hits 201 201
Misses 2 2 ☔ View full report in Codecov by Sentry. |
tested it on our dogfood instance, seems to work |
Dupe of #2919? |
Yes, but wanted an easier way of working locally with the changes. I'll merge the dupe before this one if we decide to go to 7.6.1, since that has the capability of getting rid of zookeeper |
5ca30be
to
bfe77b8
Compare
d0e0f20
to
dc44224
Compare
just use Redpanda already, lol |
f6ba6cb
to
db8bac6
Compare
9d674e7
to
ee35727
Compare
@hubertdeng123 well done! |
zookeeper_log_data_dir="/var/lib/docker/volumes/sentry-self-hosted_sentry-zookeeper-log/_data" | ||
chmod -R a+w $zookeeper_data_dir $kafka_data_dir $zookeeper_log_data_dir && returncode=$? || returncode=$? | ||
if [[ $returncode == "1" ]]; then | ||
echo "WARNING: Error when setting appropriate permissions for zookeeper, kafka, and zookeeper log docker volumes. This may corrupt your self-hosted install." |
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.
Should we point to a bug or doc with more info? It seems like users could recover from this manually if this script fails for some reason.
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.
linked to a bug to give users more context
let's try this getting this to 7.6.1 so we can get rid of zookeeper
more details: https://docs.confluent.io/platform/current/installation/migrate-zk-kraft.html