Skip to content

Commit 6402ede

Browse files
committed
fixed linting issues of avmc
1 parent 7dd0213 commit 6402ede

File tree

5 files changed

+209
-204
lines changed

5 files changed

+209
-204
lines changed

examples/odb/autonomous_vm_cluster.tf

Lines changed: 18 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -1,26 +1,27 @@
1-
# Copyright © 2025, Oracle and/or its affiliates. All rights reserved.
1+
# Copyright (c) HashiCorp, Inc.
2+
# SPDX-License-Identifier: MPL-2.0
23

34
# Autonomous VM Cluster with default maintenance window and minimum parameters
45
resource "aws_odb_cloud_autonomous_vm_cluster" "avmc_with_minimum_parameters" {
56
cloud_exadata_infrastructure_id = "<exadata_infra_id>" # refer your exadata infra id
6-
odb_network_id = "<odb_net_id>" # refer_your_odb_net_id
7+
odb_network_id = "<odb_net_id>" # refer_your_odb_net_id
78
display_name = "Ofake-avmc-my_avmc"
89
autonomous_data_storage_size_in_tbs = 5
910
memory_per_oracle_compute_unit_in_gbs = 2
1011
total_container_databases = 1
1112
cpu_core_count_per_node = 40
1213
license_model = "LICENSE_INCLUDED"
1314
# ids of db server. refer your exa infra. This is a manadatory fileld. Refer your cloud exadata infrastructure for db server id
14-
db_servers = ["<my_db_server_id>"]
15-
scan_listener_port_tls = 8561
16-
scan_listener_port_non_tls = 1024
15+
db_servers = ["<my_db_server_id>"]
16+
scan_listener_port_tls = 8561
17+
scan_listener_port_non_tls = 1024
1718
maintenance_window = {
18-
preference = "NO_PREFERENCE"
19-
days_of_week = []
20-
hours_of_day = []
21-
months = []
22-
weeks_of_month = []
23-
lead_time_in_weeks = 0
19+
preference = "NO_PREFERENCE"
20+
days_of_week = []
21+
hours_of_day = []
22+
months = []
23+
weeks_of_month = []
24+
lead_time_in_weeks = 0
2425
}
2526

2627
}
@@ -41,12 +42,12 @@ resource "aws_odb_cloud_autonomous_vm_cluster" "test" {
4142
scan_listener_port_tls = 8561
4243
scan_listener_port_non_tls = 1024
4344
maintenance_window = {
44-
preference = "CUSTOM_PREFERENCE"
45-
days_of_week = ["MONDAY", "TUESDAY"]
46-
hours_of_day = [4, 16]
47-
months = ["FEBRUARY", "MAY", "AUGUST", "NOVEMBER"]
48-
weeks_of_month = [2, 4]
49-
lead_time_in_weeks = 3
45+
preference = "CUSTOM_PREFERENCE"
46+
days_of_week = ["MONDAY", "TUESDAY"]
47+
hours_of_day = [4, 16]
48+
months = ["FEBRUARY", "MAY", "AUGUST", "NOVEMBER"]
49+
weeks_of_month = [2, 4]
50+
lead_time_in_weeks = 3
5051
}
5152
tags = {
5253
"env" = "dev"

internal/service/odb/cloud_autonomous_vm_cluster.go

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,26 +1,26 @@
1-
// Copyright © 2025, Oracle and/or its affiliates. All rights reserved.
1+
// Copyright (c) HashiCorp, Inc.
2+
// SPDX-License-Identifier: MPL-2.0
23

34
package odb
45

56
import (
67
"context"
78
"errors"
8-
"github.com/hashicorp/terraform-plugin-framework-timetypes/timetypes"
9-
"github.com/hashicorp/terraform-plugin-framework-validators/listvalidator"
10-
"github.com/hashicorp/terraform-plugin-framework/resource/schema/listplanmodifier"
119
"time"
1210

1311
"github.com/aws/aws-sdk-go-v2/aws"
1412
"github.com/aws/aws-sdk-go-v2/service/odb"
1513
odbtypes "github.com/aws/aws-sdk-go-v2/service/odb/types"
1614
"github.com/hashicorp/terraform-plugin-framework-timeouts/resource/timeouts"
15+
"github.com/hashicorp/terraform-plugin-framework-timetypes/timetypes"
16+
"github.com/hashicorp/terraform-plugin-framework-validators/listvalidator"
1717
"github.com/hashicorp/terraform-plugin-framework-validators/stringvalidator"
18-
"github.com/hashicorp/terraform-plugin-framework/path"
1918
"github.com/hashicorp/terraform-plugin-framework/resource"
2019
"github.com/hashicorp/terraform-plugin-framework/resource/schema"
2120
"github.com/hashicorp/terraform-plugin-framework/resource/schema/boolplanmodifier"
2221
"github.com/hashicorp/terraform-plugin-framework/resource/schema/float64planmodifier"
2322
"github.com/hashicorp/terraform-plugin-framework/resource/schema/int32planmodifier"
23+
"github.com/hashicorp/terraform-plugin-framework/resource/schema/listplanmodifier"
2424
"github.com/hashicorp/terraform-plugin-framework/resource/schema/planmodifier"
2525
"github.com/hashicorp/terraform-plugin-framework/resource/schema/setplanmodifier"
2626
"github.com/hashicorp/terraform-plugin-framework/resource/schema/stringplanmodifier"
@@ -509,10 +509,6 @@ func (r *resourceCloudAutonomousVmCluster) Delete(ctx context.Context, req resou
509509
}
510510
}
511511

512-
func (r *resourceCloudAutonomousVmCluster) ImportState(ctx context.Context, req resource.ImportStateRequest, resp *resource.ImportStateResponse) {
513-
resource.ImportStatePassthroughID(ctx, path.Root(names.AttrID), req, resp)
514-
}
515-
516512
func waitCloudAutonomousVmClusterCreated(ctx context.Context, conn *odb.Client, id string, timeout time.Duration) (*odbtypes.CloudAutonomousVmCluster, error) {
517513
stateConf := &retry.StateChangeConf{
518514
Pending: enum.Slice(odbtypes.ResourceStatusProvisioning),

internal/service/odb/cloud_autonomous_vm_cluster_data_source.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
1-
//Copyright © 2025, Oracle and/or its affiliates. All rights reserved.
1+
// Copyright (c) HashiCorp, Inc.
2+
// SPDX-License-Identifier: MPL-2.0
23

34
package odb
45

internal/service/odb/cloud_autonomous_vm_cluster_data_source_test.go

Lines changed: 54 additions & 51 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
1-
//Copyright © 2025, Oracle and/or its affiliates. All rights reserved.
1+
// Copyright (c) HashiCorp, Inc.
2+
// SPDX-License-Identifier: MPL-2.0
23

34
package odb_test
45

@@ -86,11 +87,8 @@ func (autonomousVMClusterDSTest) testAccCheckCloudAutonomousVmClusterDestroy(ctx
8687
}
8788
func (autonomousVMClusterDSTest) testAccPreCheck(ctx context.Context, t *testing.T) {
8889
conn := acctest.Provider.Meta().(*conns.AWSClient).ODBClient(ctx)
89-
90-
input := &odb.ListCloudAutonomousVmClustersInput{}
91-
92-
_, err := conn.ListCloudAutonomousVmClusters(ctx, input)
93-
90+
input := odb.ListCloudAutonomousVmClustersInput{}
91+
_, err := conn.ListCloudAutonomousVmClusters(ctx, &input)
9492
if acctest.PreCheckSkipError(err) {
9593
t.Skipf("skipping acceptance testing: %s", err)
9694
}
@@ -103,9 +101,10 @@ func (autonomousVMClusterDSTest) avmcBasic() string {
103101
exaInfraDisplayName := sdkacctest.RandomWithPrefix(autonomousVMClusterDSTestEntity.exaInfraDisplayNamePrefix)
104102
odbNetworkDisplayName := sdkacctest.RandomWithPrefix(autonomousVMClusterDSTestEntity.odbNetDisplayNamePrefix)
105103
avmcDisplayName := sdkacctest.RandomWithPrefix(autonomousVMClusterDSTestEntity.autonomousVmClusterDisplayNamePrefix)
106-
107-
exaInfraRes := autonomousVMClusterDSTestEntity.exaInfra(exaInfraDisplayName)
108-
odbNetRes := autonomousVMClusterDSTestEntity.odbNet(odbNetworkDisplayName)
104+
domain := acctest.RandomDomainName()
105+
emailAddress := acctest.RandomEmailAddress(domain)
106+
exaInfraRes := autonomousVMClusterDSTestEntity.exaInfra(exaInfraDisplayName, emailAddress)
107+
odbNetRes := autonomousVMClusterDSTestEntity.oracleDBNetwork(odbNetworkDisplayName)
109108
res := fmt.Sprintf(`
110109
%s
111110
@@ -116,79 +115,83 @@ data "aws_odb_db_servers_list" "test" {
116115
}
117116
118117
resource "aws_odb_cloud_autonomous_vm_cluster" "test" {
119-
cloud_exadata_infrastructure_id = aws_odb_cloud_exadata_infrastructure.test.id
120-
odb_network_id =aws_odb_network.test.id
121-
display_name = %[3]q
122-
autonomous_data_storage_size_in_tbs = 5
123-
memory_per_oracle_compute_unit_in_gbs = 2
124-
total_container_databases = 1
125-
cpu_core_count_per_node = 40
126-
license_model = "LICENSE_INCLUDED"
127-
db_servers = [ for db_server in data.aws_odb_db_servers_list.test.db_servers : db_server.id]
128-
scan_listener_port_tls = 8561
129-
scan_listener_port_non_tls = 1024
130-
maintenance_window = {
131-
preference = "NO_PREFERENCE"
132-
days_of_week = []
133-
hours_of_day = []
134-
months = []
135-
weeks_of_month =[]
136-
lead_time_in_weeks = 0
137-
}
118+
cloud_exadata_infrastructure_id = aws_odb_cloud_exadata_infrastructure.test.id
119+
odb_network_id = aws_odb_network.test.id
120+
display_name = %[3]q
121+
autonomous_data_storage_size_in_tbs = 5
122+
memory_per_oracle_compute_unit_in_gbs = 2
123+
total_container_databases = 1
124+
cpu_core_count_per_node = 40
125+
license_model = "LICENSE_INCLUDED"
126+
db_servers = [for db_server in data.aws_odb_db_servers_list.test.db_servers : db_server.id]
127+
scan_listener_port_tls = 8561
128+
scan_listener_port_non_tls = 1024
129+
maintenance_window {
130+
custom_action_timeout_in_mins = 16
131+
is_custom_action_timeout_enabled = true
132+
patching_mode = "ROLLING"
133+
preference = "NO_PREFERENCE"
134+
}
138135
139136
}
140137
141138
142139
data "aws_odb_cloud_autonomous_vm_cluster" "test" {
143-
id = aws_odb_cloud_autonomous_vm_cluster.test.id
140+
id = aws_odb_cloud_autonomous_vm_cluster.test.id
144141
145142
}
146143
`, exaInfraRes, odbNetRes, avmcDisplayName)
147144

148145
return res
149146
}
150147

151-
func (autonomousVMClusterDSTest) odbNet(odbNetName string) string {
148+
func (autonomousVMClusterDSTest) oracleDBNetwork(odbNetName string) string {
152149
networkRes := fmt.Sprintf(`
153150
154151
152+
153+
155154
resource "aws_odb_network" "test" {
156-
display_name = %[1]q
155+
display_name = %[1]q
157156
availability_zone_id = "use1-az6"
158157
client_subnet_cidr = "10.2.0.0/24"
159158
backup_subnet_cidr = "10.2.1.0/24"
160-
s3_access = "DISABLED"
161-
zero_etl_access = "DISABLED"
159+
s3_access = "DISABLED"
160+
zero_etl_access = "DISABLED"
162161
}
163162
163+
164164
`, odbNetName)
165165
return networkRes
166166
}
167167

168-
func (autonomousVMClusterDSTest) exaInfra(exaInfraName string) string {
168+
func (autonomousVMClusterDSTest) exaInfra(exaInfraName, emailAddress string) string {
169169
exaInfraRes := fmt.Sprintf(`
170170
171171
172+
173+
172174
resource "aws_odb_cloud_exadata_infrastructure" "test" {
173-
display_name = %[1]q
174-
shape = "Exadata.X9M"
175-
storage_count = 3
176-
compute_count = 2
177-
availability_zone_id = "use1-az6"
178-
customer_contacts_to_send_to_oci = ["[email protected]"]
179-
maintenance_window = {
180-
custom_action_timeout_in_mins = 16
181-
days_of_week = []
182-
hours_of_day = []
183-
is_custom_action_timeout_enabled = true
184-
lead_time_in_weeks = 0
185-
months = []
186-
patching_mode = "ROLLING"
187-
preference = "NO_PREFERENCE"
188-
weeks_of_month =[]
175+
display_name = %[1]q
176+
shape = "Exadata.X9M"
177+
storage_count = 3
178+
compute_count = 2
179+
availability_zone_id = "use1-az6"
180+
customer_contacts_to_send_to_oci = ["%[2]s"]
181+
maintenance_window = {
182+
custom_action_timeout_in_mins = 16
183+
days_of_week = []
184+
hours_of_day = []
185+
is_custom_action_timeout_enabled = true
186+
lead_time_in_weeks = 0
187+
months = []
188+
patching_mode = "ROLLING"
189+
preference = "NO_PREFERENCE"
190+
weeks_of_month = []
189191
}
190192
}
191193
192-
`, exaInfraName)
194+
195+
`, exaInfraName, emailAddress)
193196
return exaInfraRes
194197
}

0 commit comments

Comments
 (0)