cluster-api-provider-cloud-director
cluster-api-provider-cloud-director copied to clipboard
CAFV-393: Do not update VAppMetadataUpdated anymore
Description
Please provide a brief description of the changes proposed in this Pull Request
- This stops updating the vAppMetadataUpdated field. The VCDResourceSet already contains the list of vApps consumed by the cluster. This field is not needed anymore.
- This does not remove any race scenarios.
- This field can remove the number of VCD API calls while a cluster is being created by 1 call per reentry to the reconciler. That is a small tax when the number of calls is considered. This can be reduced by other means as well.
Checklist
- [X] tested locally
- [ ] updated any relevant dependencies
- [ ] updated any relevant documentation or examples
API Changes
Are there API changes?
- [X] Yes
- [ ] No
If yes, please fill in the below
- Updated conversions?
- [ ] Yes
- [ ] No
- [X] N/A
- Updated CRDs?
- [ ] Yes
- [ ] No
- [X] N/A
- Updated infrastructure-components.yaml?
- [ ] Yes
- [ ] No
- [X] N/A
- Updated
./examples/capi-quickstart.yaml
?- [ ] Yes
- [ ] No
- [X] N/A
- Updated necessary files under
./infrastructure-vcd/v1.0.0/
?- [ ] Yes
- [ ] No
- [X] N/A
Issue
If applicable, please reference the relevant issue
Fixes #