cluster-api-provider-cloud-director icon indicating copy to clipboard operation
cluster-api-provider-cloud-director copied to clipboard

Add `node.kubernetes.io/instance-type` label to nodes

Open viveksyngh opened this issue 1 year ago • 3 comments

Is your feature request related to a problem? Please describe.

We would like to add node.kubernetes.io/instance-type labels to nodes provisioned using CAPVCD similar to other cloud provider node.kubernetes.io/instance-type.

https://kubernetes.io/docs/reference/labels-annotations-taints/#nodekubernetesioinstance-type

Describe the solution you'd like

We can add the sizing policy of VCDMachine object as instanceType of the nodes. These labels can be added as metadata in init or join configuration of the nodes.

Describe alternatives you've considered

No response

Additional context

No response

viveksyngh avatar Nov 01 '23 09:11 viveksyngh

As far as I understand, the instance-type label is set by kubelet, based on information it retrieves using the InstanceType method of the Cloud Provider Interface (CPI).

For example, the vSphere Cloud Provider implements this method.

I think this issue should be moved to the VCD Cloud Provider, https://github.com/vmware/cloud-provider-for-cloud-director

dlipovetsky avatar Nov 01 '23 15:11 dlipovetsky

Hi @viveksyngh can you please open the issue in CPI project as the changes need to be done in https://github.com/vmware/cloud-provider-for-cloud-director repository ?

Thank you!

Anirudh9794 avatar Nov 01 '23 21:11 Anirudh9794

Sure

viveksyngh avatar Nov 02 '23 10:11 viveksyngh