-
Notifications
You must be signed in to change notification settings - Fork 157
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
Enhancement proposal for replacing nodes using their HostIDs #1313
Conversation
8da0f88
to
011c5cf
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.
A few suggestions and comments, lgtm otherwise
enhancements/proposals/1311-replace-nodes-using-hostid/README.md
Outdated
Show resolved
Hide resolved
enhancements/proposals/1311-replace-nodes-using-hostid/README.md
Outdated
Show resolved
Hide resolved
enhancements/proposals/1311-replace-nodes-using-hostid/README.md
Outdated
Show resolved
Hide resolved
enhancements/proposals/1311-replace-nodes-using-hostid/README.md
Outdated
Show resolved
Hide resolved
enhancements/proposals/1311-replace-nodes-using-hostid/README.md
Outdated
Show resolved
Hide resolved
enhancements/proposals/1311-replace-nodes-using-hostid/README.md
Outdated
Show resolved
Hide resolved
enhancements/proposals/1311-replace-nodes-using-hostid/README.md
Outdated
Show resolved
Hide resolved
enhancements/proposals/1311-replace-nodes-using-hostid/README.md
Outdated
Show resolved
Hide resolved
011c5cf
to
460206a
Compare
93fbe68
to
3baab9a
Compare
Currently, the Scylla Operator uses IP addresses of a node to identify which node should be replaced. The issue is that depending on the ScyllaCluster configuration, IP addresses might be ephemeral so it’s error prone to use it as an identifier as they may change during when the procedure is executed. Because there is a better stable alternative, IP address approach became deprecated, hence we need to shift to it. Details can be found in actual enhancement.
3baab9a
to
27fac54
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
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: tnozicka, zimnx The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Currently, the Scylla Operator uses IP addresses of a node to identify which node should be replaced. The issue is that depending on the ScyllaCluster configuration, IP addresses might be ephemeral so it’s error prone to use it as an identifier as they may change during when the procedure is executed. Because there is a better stable alternative, IP address approach became deprecated, hence we need to shift to it.
Details can be found in actual enhancement.
Fixes #1314