kubernetes-sigs / cluster-api

Home for Cluster API, a subproject of sig-cluster-lifecycle
https://cluster-api.sigs.k8s.io
Apache License 2.0
3.5k stars 1.3k forks source link

[question] do we support external cloud provider ? #2977

Closed jichenjc closed 4 years ago

jichenjc commented 4 years ago

User Story This comes from https://github.com/kubernetes-sigs/cluster-api-provider-openstack/issues/489

openstack used to have intree cloud provider and now it's targeting to move to external and some projects like https://github.com/kubernetes/cloud-provider-openstack will be used,

so the question is do we need to consider this into 1) a general cluster-api consideration 2) an addon to cluster-api 3) need each cluster api provider consider it?

Detailed Description

[A clear and concise description of what you want to happen.]

Anything else you would like to add:

[Miscellaneous information that will assist in solving the issue.]

/kind feature

detiber commented 4 years ago

@jichenjc There should be nothing preventing the use of the in-treeout-of-tree cloud provider that I'm aware of. I know https://github.com/kubernetes-sigs/cluster-api-provider-vsphere makes use of it and cluster-api-provider-azure recently added support as well: https://github.com/kubernetes-sigs/cluster-api-provider-azure/issues/456

ncdc commented 4 years ago

@detiber I assume you meant out-of-tree 😄

detiber commented 4 years ago

doh, yes, edited the comment to fix :joy:

jichenjc commented 4 years ago

@detiber thanks a lot :) yes, I mean out of tree provider so it's provider's responsibility to create those and not in cluster-api scope :)

thanks ~

@sbueringer FYI

sbueringer commented 4 years ago

So in Azure it seems to be just another flavor and a document describing how to deploy the CCM: https://github.com/kubernetes-sigs/cluster-api-provider-azure/pull/527/files

Seems reasonable

jichenjc commented 4 years ago

yes, so this is something we can replicate into ours not something in cluster-api scope, instead, do it in provider..