cluster-api-provider-vsphere icon indicating copy to clipboard operation
cluster-api-provider-vsphere copied to clipboard

Remove the tagging functionality of the VSphereFailureDomain API

Open srm09 opened this issue 2 years ago • 8 comments

/kind feature

Describe the solution you'd like Traditionally VI admins have been responsible for tagging and managing the vCenter resources. Having CAPV tag resources out of band is a departure from this behavior. From the feedback we received from our users, this is a functionality that they do not want to expose to CAPV, but have the VI admins perform resource tagging as a pre-requisite.

Anything else you would like to add: Deprecate the functionality via the following steps:

  • Introduce the deprecation notice on the API type
  • Remove the controller logic for resource tagging

Environment:

  • Cluster-api-provider-vsphere version:
  • Kubernetes version: (use kubectl version):
  • OS (e.g. from /etc/os-release):

srm09 avatar Apr 28 '23 15:04 srm09

Discussed with Sagar, some additional information:

  • autoConfigure takes care of creating the tags/categories, but does not delete them on cleanup
  • once the tag/category is created, it can be used by other resources in VC, so deleting it might not lead to the best outcome

sbueringer avatar Jul 28 '23 09:07 sbueringer

/retitle Remove the tagging functionality of the VSphereFailureDomain

We'll use this issue to track the deprecation and removal. PR for deprecation is open: https://github.com/kubernetes-sigs/cluster-api-provider-vsphere/pull/2103

/kind release-blocking (for the deprecation part)

sbueringer avatar Jul 28 '23 09:07 sbueringer

/retitle Remove the tagging functionality of the VSphereFailureDomain API

sbueringer avatar Jul 28 '23 09:07 sbueringer

/lifecycle frozen

randomvariable avatar Jul 28 '23 16:07 randomvariable

/kind api-change

sbueringer avatar Jul 28 '23 16:07 sbueringer

/remove-kind release-blocking

sbueringer avatar Jul 31 '23 15:07 sbueringer

Note: We should also update the failure domain proposal to simplify it once the auto configure field was removed

sbueringer avatar Aug 01 '23 07:08 sbueringer

Field is now deprecated, will be removed with v1beta2

sbueringer avatar Aug 18 '23 12:08 sbueringer