-
Notifications
You must be signed in to change notification settings - Fork 6.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
[etcd] Sometimes, we do not need to run etcd role on all nodes. #9173
[etcd] Sometimes, we do not need to run etcd role on all nodes. #9173
Conversation
8acdc34
to
b7c3e19
Compare
d941ade
to
ae89cde
Compare
Nice cleanup/speedup @liupeng0518 /kind feature |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cristicalin, liupeng0518 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
ae89cde
to
63cf42c
Compare
8dc1dd7
to
a14fc9e
Compare
Hi @liupeng0518, please rebase this PR on the current state of the master branch to be able to pass the CI tests. |
a14fc9e
to
4da26c4
Compare
/lgtm |
@jayonlau: changing LGTM is restricted to collaborators In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/lgtm |
@jayonlau: changing LGTM is restricted to collaborators In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/lgtm |
/test ? |
@liupeng0518: The following commands are available to trigger required jobs:
Use In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/test all |
…rnetes-sigs#9173) * WIP: sometimes,we not run etcd * fix ansible lint * like calico(kdd) cni, no need run etcd
…rnetes-sigs#9173) * WIP: sometimes,we not run etcd * fix ansible lint * like calico(kdd) cni, no need run etcd
…rnetes-sigs#9173) * WIP: sometimes,we not run etcd * fix ansible lint * like calico(kdd) cni, no need run etcd
What type of PR is this?
What this PR does / why we need it:
Some plugins, we do not need to run etcd role on all nodes.
It only needs to be run at the etcd node and the control node.
similar kubeadm-etcd:
kubespray/roles/kubernetes/kubeadm/tasks/main.yml
Lines 155 to 161 in 8585134
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?: