Skip to content
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

Vdc bug fixes for disabling storage profile and deserializing type.Task on vdc.update #433

Merged
merged 30 commits into from
Feb 21, 2022

Conversation

vbauzys
Copy link
Contributor

@vbauzys vbauzys commented Feb 2, 2022

Ref:vmware/terraform-provider-vcd#754
Ref:#431

Fixes Issue "Wrong order in Task structure" for VDC update
Fixes Issue where VDC creation with storage profile enabled=false wasn't working

vbauzys and others added 26 commits August 25, 2020 08:55
Merge branch 'master' of github.com:vmware/go-vcloud-director
Merge branch 'master' of github.com:vmware/go-vcloud-director
Merge branch 'master' of github.com:vmware/go-vcloud-director
# Conflicts:
#	govcd/api.go
#	govcd/api_vcd_test.go
#	govcd/openapi_endpoints.go
#	types/v56/constants.go
#	types/v56/openapi.go
@vbauzys vbauzys self-assigned this Feb 2, 2022
@vbauzys vbauzys marked this pull request as ready for review February 2, 2022 15:11
@vbauzys vbauzys requested review from mikeletux and adambarreiro and removed request for dataclouder February 2, 2022 15:11
@vbauzys vbauzys changed the title Vdc fixes Vdc bug fixes for disabling storage profile and deserializing task in VDC task history Feb 2, 2022
@vbauzys vbauzys changed the title Vdc bug fixes for disabling storage profile and deserializing task in VDC task history Vdc bug fixes for disabling storage profile and deserializing type.Task on vdc.update Feb 2, 2022
@@ -0,0 +1,2 @@
* Fixes Issue #431 "Wrong order in Task structure" [GH-433]
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IMO we need to mention "convert field types" - so that it is clear why we broke compatibility

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

added comment

@vbauzys vbauzys merged commit d0450d3 into vmware:main Feb 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants