-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Refactor to call non-voting servers read replicas #9191
Conversation
12aef88
to
ad12d29
Compare
ad12d29
to
861adee
Compare
861adee
to
cb26476
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
cb26476
to
4d777f3
Compare
eebc9be
to
51f40b7
Compare
This hopefully will prevent some confusion and also make it clearer that redundancy zones don’t rely on non voting servers.
Co-authored-by: Kit Patella <[email protected]>
51f40b7
to
3e6c17c
Compare
🍒 If backport labels were added before merging, cherry-picking will start automatically. To retroactively trigger a backport after merging, add backport labels and re-run https://circleci.com/gh/hashicorp/consul/283524. |
🍒✅ Cherry pick of commit 66fd23d onto |
Co-authored-by: Kit Patella <[email protected]>
This hopefully will prevent some confusion and also make it clearer that redundancy zones don’t rely on non voting servers.
Most of this is a simple refactoring of variable names and changing docs. There are a number of deprecations of old names in favor of the new naming. In all cases (serf tags, cli flags, configuration settings, and
consul
service meta) both the old and new names will coexist together and the deprecation notices say that the old names will be removed in a future release.