-
Notifications
You must be signed in to change notification settings - Fork 715
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
Specify node taint on node join #209
Comments
I'm working on a PR on this and #202 |
See my comment: #202 (comment) |
Issues go stale after 90d of inactivity. Prevent issues from auto-closing with an If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or |
Issues go stale after 90d of inactivity. Prevent issues from auto-closing with an If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or |
1 similar comment
Issues go stale after 90d of inactivity. Prevent issues from auto-closing with an If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or |
/remove-lifecycle stale |
/assign @fabriziopandini @timothysc @fabriziopandini Is there anything left todo. |
@timothysc this is still not addressed. Following options were considered and then discarded:
The only option left on the table is to wait for the node to join and then send labels to the apiserver, but it is kind of risky for worker nodes - were the only identity you get is the token - (see #202 (comment) from JBeda). IMO we can create all the necessary facilities and the let the user explicitly opt-in:
WDYT? |
So the auth folks have ki-boshed this idea as a sec-hole. I think we're going to have to admin control. |
Similar to #202, we should have an option to taint a node on join. One use case is when a cluster would have a set of nodes dedicated for ingress-controllers. A DaemonSet could be deployed that can use the label option in #202, and the taint option here. Otherwise this is a manual process.
The text was updated successfully, but these errors were encountered: