-
Notifications
You must be signed in to change notification settings - Fork 500
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
Set the default externalTrafficPolicy to be Local for TiDB service #947
Set the default externalTrafficPolicy to be Local for TiDB service #947
Conversation
Can we do this in the terraform deployment script? This looks like a breaking change to the operator. |
@weekface: which part/file do you have in mind for "the terraform deployment script"? |
@shinnosuke-okada The default values.yaml for terraform module https://github.com/pingcap/tidb-operator/blob/master/deploy/modules/aws/tidb-cluster/values/default.yaml |
@aylei PTAL |
Users may deploy tidb-cluster in their IDC where a node can have multiple pods scheduled. |
c1dd6af
to
d8eeffc
Compare
Addressed the review comment. PTAL. |
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
/run-e2e-in-kind |
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
cherry pick to release-1.0 in PR #960 |
Signed-off-by: Ran <huangran@pingcap.com>
What problem does this PR solve?
#879
What is changed and how does it work?
The default externalTrafficPolicy is now set to be Local
Check List
Tests
kubectl describe service my-cluster-tidb -n my-cluster
and verify the value of externalTrafficPolicy.Code changes
Side effects
Related changes
Does this PR introduce a user-facing change?: