-
Notifications
You must be signed in to change notification settings - Fork 112
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
Add vcd_ui_plugin resource and data source to programmatically manage UI Plugins #1059
Conversation
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
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.
I have tested this manually and it works good but have one concern about update (inline)
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
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.
I have tried manually working with plugin as well as ran tests on 10.3.0, 10.4.0, 10.4.1
examples/container-service-extension-4.0/install/step2/terraform.tfvars.example
Outdated
Show resolved
Hide resolved
examples/container-service-extension-4.0/install/step2/terraform.tfvars.example
Outdated
Show resolved
Hide resolved
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
Signed-off-by: abarreiro <[email protected]>
Overview
This PR adds a new resource and data source vcd_ui_plugin, so service providers can install UI Plugins programmatically.
Despite it can be a bit contradictory at first glance, as Terraform is a pure IaC tool, it makes sense in a scenario where the service provider uses Terraform to automate all VCD infrastructure but her tenant users don't. In this case, a service provider may need to install the UI Plugins using Terraform for their tenants to use them.
This PR additionally updates the CSE guide to install the Kubernetes Container Clusters UI Plugin making use of this new resource.
Depdendencies
Requires vmware/go-vcloud-director#575
Tests
Tested on VCD 10.4.2, 10.4.1, 10.3.3, 10.3.0