-
Notifications
You must be signed in to change notification settings - Fork 39
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
managed control planes config patch management #725
Comments
This should be done by the infrastructure provider via injecting appropriate MachineConfig patch (as the provider knows the endpoints, and it makes sense always to have them). |
Implementation of this feature might be more tricky than expected. Creating services is not required for the regular KubeVirt provider, it feels like a thing more specific to the managed control planes only. So there are two options Create a forked version of th KubeVirt providerThen implement the config patch and service creation flow there. Create a separate service to handle configsThe separate service will be very similar to the infra providers, but will handle only config patches creation. |
Can you help me understand what the service is needed for? Do nodes need to be publicly accessible for KubeSpan? |
I don't think it is specific to managed control planes, but rather a feature flag for the KubeVirt provider to establish a "hole" which KubeSpan can use. It should be enabled for managed control planes case, but might be useful outside of it. So I would rather add it as a feature flag (can even be a machine request property?). |
If we agree to bring more complexity to the infra provider, I have no objections against that. |
The way we plan to set up KubeVirt we need to expose the nodes through the kubernetes services, and we need the additional config to tell the other members about it. |
Implement the controller that will create high priority config patches for the cluster which is using managed control planes.
We need config patches for the following:
The provider should be able to:
The text was updated successfully, but these errors were encountered: