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

flush servers #104

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Conversation

summerainxiao
Copy link

When connecting to the service domain name with k8s, the client connection fails due to the IP change after zookeeper restarts. So you need to refresh the servers

jeffbean and others added 9 commits August 1, 2019 13:36
* Setting up travis and builds for more than one ZK version to test against
* Reorg tests to support 3.4 and 3.5 clusters
* travis up to go 1.11
* Adding more go version to the matrix and allow failures for go tip and 3.5 zk
* Change to always using zookeeper server scripts. Adding wait for start in test scripts
* Adding waitForStop on testing helpers. Also update comments
* Light test cleanup - more makefile to start to have a common set of build instructions
* ZK 3.5 part 2 - Adding reconfig and incremental reconfig apis
* adjust test config from 3.5 option - dont ignore 3.5 test matrix
* test cases dont care if we get a connection close as we might have been connected to a node that is no longer accepting connecitons
Because these files had exported functions it's possible someone is using them, but it's not good for non test files to import testing. If it turns out these are used outside of this repo then they can be moved to another sub-package that can be imported when needed.

Fixes samuel#217
Adding power support
@GEverding
Copy link

Is there any more work on this?

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.

5 participants