-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Allow multiple etcd host #8845
Allow multiple etcd host #8845
Conversation
Important Review skippedAuto reviews are limited to specific labels. Labels to auto review (1)
Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configuration File (
|
1b0056d
to
80df527
Compare
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.
Thank you for the PR!
Please refer to our contribution guidelines on how to create commit messages.
This will also need a release note.
80df527
to
52f13ad
Compare
Signed the commit. i dont know what to write to release note. |
Can be something simple, like "Allow multiple etcd hosts to be specified in |
f759d5d
to
6676256
Compare
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 for the PR!
The changes here are made in a Golang submodule (kvdb
, see go.mod), which means this will need another PR that updates the submodule in the main go.mod
file.
This can't be done in the same PR, mostly just FYI.
So this looks good to me once we update the configuration description (so users know this is a feature).
@bitbandi, remember to re-request review from reviewers when ready |
5f9f7d4
to
04871c2
Compare
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, thanks for the fix!
04871c2
to
0bbb730
Compare
0bbb730
to
6043ced
Compare
conflict resolved |
Change Description
This patch allow set multiple etcd servers. like an etcd cluster in docker swarm/kubernetes.
Steps to Test
Pull Request Checklist
Testing
Code Style and Documentation
[skip ci]
in the commit message for small changes.📝 Please see our Contribution Guidelines for further guidance.