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

Fixes #1322

Merged
merged 11 commits into from
Sep 13, 2024
Merged

Fixes #1322

Show file tree
Hide file tree
Changes from 9 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions .changes/v3.14.0/1295-features.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
* **New Resource:** `vcd_external_endpoint` to manage External Endpoints [GH-1295]
* **New Data Source:** `vcd_external_endpoint` to read External Endpoints [GH-1295]
* **New Resource:** `vcd_api_filter` to manage API Filters [GH-1295]
* **New Data Source:** `vcd_api_filter` to read API Filters [GH-1295]
* **New Resource:** `vcd_external_endpoint` to manage External Endpoints [GH-1295, GH-1322]
* **New Data Source:** `vcd_external_endpoint` to read External Endpoints [GH-1295, GH-1322]
* **New Resource:** `vcd_api_filter` to manage API Filters [GH-1295, GH-1322]
* **New Data Source:** `vcd_api_filter` to read API Filters [GH-1295, GH-1322]
2 changes: 1 addition & 1 deletion .changes/v3.14.0/1296-improvements.md
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
* Add `vcd_nsxt_alb_edgegateway_service_engine_group` and `vcd_nsxt_alb_service_engine_group` resource types to
`vcd_resource_list` data source [GH-1296]
`vcd_resource_list` data source [GH-1296, GH-1322]
2 changes: 1 addition & 1 deletion .changes/v3.14.0/1297-improvements.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
* Add support for NSX-T Non-distributed Org VDC networks via flags `non_distributed_routing_enabled`
in `vcd_nsxt_edgegateway` and `interface_type=non_distributed` in `vcd_network_routed_v2`
[GH-1297]
[GH-1297, GH-1322]
1 change: 1 addition & 0 deletions .changes/v3.14.0/1322-notes.md
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
* Corrected a malformed HCL snippet in the *"Connecting with a Service Account API token"* section of the documentation [GH-1322]
1 change: 1 addition & 0 deletions scripts/skip-upgrade-tests.txt
Original file line number Diff line number Diff line change
Expand Up @@ -358,3 +358,4 @@ vcd.ResourceSchema-vcd_rde_interface_behavior.tf v3.13.0 "Changed fields to comp
vcd.ResourceSchema-vcd_rde_type_behavior.tf v3.13.0 "Changed fields to computed and added new arguments"
vcd.ResourceSchema-vcd_vm_sizing_policy.tf v3.13.0 "Changed description"
vcd.ResourceSchema-vcd_vm_vgpu_policy.tf v3.13.0 "Changed description"
vcd.ResourceSchema-vcd_network_routed_v2.tf v3.13.0 "Changed description"
2 changes: 1 addition & 1 deletion vcd/datasource_vcd_resource_list_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,6 @@ func TestAccVcdDatasourceResourceList(t *testing.T) {
{name: "global_role", resourceType: "vcd_global_role", knownItem: "vApp Author"},
{name: "rights_bundle", resourceType: "vcd_rights_bundle", knownItem: "Default Rights Bundle"},
{name: "right", resourceType: "vcd_right", knownItem: "Catalog: Change Owner"},
{name: "alb-service-engine-group", resourceType: "vcd_nsxt_alb_service_engine_group"},

// entities belonging to an Org don't require an explicit parent, as it is given from the Org passed in the provider
// For each resource, we test with and without and explicit parent
Expand All @@ -43,6 +42,7 @@ func TestAccVcdDatasourceResourceList(t *testing.T) {

if usingSysAdmin() {
lists = append(lists, listDef{name: "admin-vdc-template", resourceType: "vcd_org_vdc_template"})
lists = append(lists, listDef{name: "alb-service-engine-group", resourceType: "vcd_nsxt_alb_service_engine_group"})
}

knownNetworkPool1 := testConfig.VCD.ProviderVdc.NetworkPool
Expand Down
4 changes: 4 additions & 0 deletions vcd/resource_vcd_api_filter_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,10 @@ func TestAccVcdApiFilter(t *testing.T) {
preTestChecks(t)
skipIfNotSysAdmin(t)

if checkVersion(testConfig.Provider.ApiVersion, "< 37.3") {
t.Skipf("This test tests VCD 10.4.3+ (API V37.3+) features. Skipping.")
}

var params = StringMap{
"Vendor": "vmware",
"Name": t.Name(),
Expand Down
4 changes: 4 additions & 0 deletions vcd/resource_vcd_external_endpoint_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,10 @@ func TestAccVcdExternalEndpoint(t *testing.T) {
preTestChecks(t)
skipIfNotSysAdmin(t)

if checkVersion(testConfig.Provider.ApiVersion, "< 37.3") {
t.Skipf("This test tests VCD 10.4.3+ (API V37.3+) features. Skipping.")
}

vendor := "vmware"
name := t.Name()
version := "1.0.0"
Expand Down
1 change: 1 addition & 0 deletions vcd/resource_vcd_network_routed_v2_nsxt_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -1135,6 +1135,7 @@ func TestAccVcdRoutedNetworkV2MetadataIgnore(t *testing.T) {

func TestAccVcdRoutedNetworkV2NonDistributed(t *testing.T) {
preTestChecks(t)
skipIfNotSysAdmin(t)

// String map to fill the template
var params = StringMap{
Expand Down
1 change: 0 additions & 1 deletion website/docs/index.html.markdown
Original file line number Diff line number Diff line change
Expand Up @@ -233,7 +233,6 @@ and activating a VCD Service Account can be found in the
provider "vcd" {
auth_type = "service_account_token_file"
service_account_token_file = "token.json"
sysorg = "System"
org = var.vcd_org # Default for resources
vdc = var.vcd_vdc # Default for resources
url = var.vcd_url
Expand Down
Loading