-
Notifications
You must be signed in to change notification settings - Fork 113
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
Update ALB service engine group and ALB settings to add supported_feature_set for VCD 10.4 #878
Conversation
Signed-off-by: abarreiro <abarreiro@vmware.com>
Signed-off-by: abarreiro <abarreiro@vmware.com>
Signed-off-by: abarreiro <abarreiro@vmware.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Few small asks.
Signed-off-by: abarreiro <abarreiro@vmware.com>
Signed-off-by: abarreiro <abarreiro@vmware.com>
Signed-off-by: abarreiro <abarreiro@vmware.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Signed-off-by: abarreiro <abarreiro@vmware.com>
Signed-off-by: abarreiro <abarreiro@vmware.com>
Signed-off-by: abarreiro <abarreiro@vmware.com>
…o/terraform-provider-vcd into update-avi-licensing Signed-off-by: abarreiro <abarreiro@vmware.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left a few minor comments which need addressing. We should be good after that.
Signed-off-by: abarreiro <abarreiro@vmware.com>
Signed-off-by: abarreiro <abarreiro@vmware.com>
This PR is not associated to any issue.
This PR depends on vmware/go-vcloud-director#485
Description
When executing the tests using VCD 10.4, I found that some of them were failing due to:
license_type
attribute from 10.4 (v37.0) onwardssupported_feature_set
attribute to be compatible with 10.4 (v37.0) that replaces the abovelicense_type
supported_feature_set
attribute to be compatible with 10.4 (v37.0) that replaces the abovelicense_type
The goal of this PR is to adapt the mentioned resources and data sources to make the Terraform Provider compatible with v37.0 of the API (VCD 10.4).
In consequence, the failing tests have been adapted and resources and datasources have changed the mentioned required/optional attributes.
Tests
alb
tag passes on any VCD