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

Bump docker-compose ZooKeeper version to 3.5.5 to fix compatability bug. #606

Closed
wants to merge 1 commit into from

Conversation

jpittis
Copy link

@jpittis jpittis commented Dec 13, 2023

Context

#605 describes a fatal issue with the existing docker-compose file that I resolved with the contents of this PR. Based on conversations in yahoo/CMAK#748, it sounds like Curator is no longer compatible with ZooKeeper 3.4.

How to reproduce and test

The following commands will reproduce the issue, which can then be resolved by using this commit instead:

$ mkdir mantis
$ cd mantis
$ wget https://raw.githubusercontent.com/Netflix/mantis/master/docker-compose.yml
$ docker-compose -f docker-compose.yml up

You'll likely need to prune your volumes when switching between versions to allow for the ZooKeeper upgrade.

Checklist

Based on conversations in yahoo/CMAK#748, it sounds
like Curator is no longer compatible with ZooKeeper 3.4.
@jpittis
Copy link
Author

jpittis commented Dec 13, 2023

Closing as per #605. Helm should be used instead of docker-compose. We should update the docs to point to helm, and remove these outdated docker-compose files.

@jpittis jpittis closed this Dec 13, 2023
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.

1 participant