Skip to content

Commit 32a612c

Browse files
Terraform Team Automationravinitp
authored andcommitted
Bug Fix: Fixed DB unique name parameter not getting honoured for new base DB system resource
1 parent a9ce84b commit 32a612c

File tree

4 files changed

+24
-12
lines changed

4 files changed

+24
-12
lines changed

examples/database/db_systems/db_vm/db_vm_amd/main.tf

Lines changed: 7 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ variable "db_admin_password" {
5555
}
5656

5757
variable "db_version" {
58-
default = "19.0.0.0"
58+
default = "19.24.0.0"
5959
}
6060

6161
variable "db_disk_redundancy" {
@@ -146,17 +146,13 @@ data "oci_database_databases" "databases" {
146146
db_home_id = data.oci_database_db_homes.db_homes.db_homes[0].db_home_id
147147
}
148148

149-
d
150-
151-
152-
153-
ata "oci_database_db_versions" "test_db_versions_by_db_system_id" {
149+
data "oci_database_db_versions" "test_db_versions_by_db_system_id" {
154150
compartment_id = var.compartment_ocid
155151
db_system_id = oci_database_db_system.test_db_system.id
156152
}
157153

158154
resource "oci_database_backup" "test_backup" {
159-
database_id = "${data.oci_database_databases.databases.databases.0.id}"
155+
database_id = data.oci_database_databases.databases.databases.0.id
160156
display_name = "Monthly Backup"
161157
}
162158

@@ -303,6 +299,7 @@ resource "oci_database_db_system" "test_db_system" {
303299
# kms_key_id = var.kms_key_id
304300
# vault_id = var.vault_id
305301
db_name = "aTFdbVm"
302+
db_unique_name = "aTFdbVm_xyz"
306303
character_set = var.character_set
307304
ncharacter_set = var.n_character_set
308305
db_workload = var.db_workload
@@ -313,7 +310,7 @@ resource "oci_database_db_system" "test_db_system" {
313310
}
314311
}
315312

316-
db_version = "19.15.0.0"
313+
db_version = "19.24.0.0"
317314
display_name = "MyTFDBHomeVm"
318315
}
319316

@@ -360,12 +357,12 @@ resource "oci_database_db_system" "db_system_bkup" {
360357
display_name = "tfDbSystemFromBackupWithCustImg"
361358

362359
db_home {
363-
db_version = "19.15.0.0"
360+
db_version = "19.24.0.0"
364361
# database_software_image_id = var.test_database_software_image_ocid
365362
database {
366363
admin_password = "BEstrO0ng_#11"
367364
backup_tde_password = "BEstrO0ng_#11"
368-
backup_id = "${oci_database_backup.test_backup.id}"
365+
backup_id = oci_database_backup.test_backup.id
369366
db_name = "dbback"
370367
}
371368
}

internal/integrationtest/database_db_system_resource_amd_vm_test.go

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,11 +30,12 @@ func TestResourceDatabaseDBSystemAmdVM(t *testing.T) {
3030
node_count = "1"
3131
display_name = "tfDbSystemTest"
3232
db_home {
33-
db_version = "19.15.0.0"
33+
db_version = "19.24.0.0"
3434
display_name = "dbHome1"
3535
database {
3636
admin_password = "BEstrO0ng_#11"
3737
db_name = "aTFdb"
38+
db_unique_name = "aTFdb_xyz"
3839
character_set = "AL32UTF8"
3940
ncharacter_set = "AL16UTF16"
4041
db_workload = "OLTP"
@@ -76,8 +77,9 @@ func TestResourceDatabaseDBSystemAmdVM(t *testing.T) {
7677
resource.TestCheckResourceAttr(ResourceDatabaseResourceName, "data_storage_size_in_gb", "256"),
7778
resource.TestCheckResourceAttr(ResourceDatabaseResourceName, "license_model", "LICENSE_INCLUDED"),
7879
resource.TestCheckResourceAttr(ResourceDatabaseResourceName, "node_count", "1"),
79-
resource.TestCheckResourceAttr(ResourceDatabaseResourceName, "db_home.0.db_version", "19.15.0.0"),
80+
resource.TestCheckResourceAttr(ResourceDatabaseResourceName, "db_home.0.db_version", "19.24.0.0"),
8081
resource.TestCheckResourceAttrSet(ResourceDatabaseResourceName, "db_home.0.display_name"),
82+
resource.TestCheckResourceAttr(ResourceDatabaseResourceName, "db_home.0.database.0.db_unique_name", "aTFdb_xyz"),
8183
),
8284
},
8385
})

internal/service/database/database_db_system_resource.go

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -301,7 +301,9 @@ func DatabaseDbSystemResource() *schema.Resource {
301301
},
302302
"db_unique_name": {
303303
Type: schema.TypeString,
304+
Optional: true,
304305
Computed: true,
306+
ForceNew: true,
305307
},
306308
"id": {
307309
Type: schema.TypeString,
@@ -1551,6 +1553,11 @@ func (s *DatabaseDbSystemResourceCrud) mapToCreateDatabaseDetails(fieldKeyFormat
15511553
result.DbName = &tmp
15521554
}
15531555

1556+
if dbUniqueName, ok := s.D.GetOkExists(fmt.Sprintf(fieldKeyFormat, "db_unique_name")); ok {
1557+
tmp := dbUniqueName.(string)
1558+
result.DbUniqueName = &tmp
1559+
}
1560+
15541561
if dbWorkload, ok := s.D.GetOkExists(fmt.Sprintf(fieldKeyFormat, "db_workload")); ok {
15551562
result.DbWorkload = oci_database.CreateDatabaseDetailsDbWorkloadEnum(dbWorkload.(string))
15561563
}
@@ -1623,6 +1630,10 @@ func CreateDatabaseDetailsToMap(obj *oci_database.CreateDatabaseDetails) map[str
16231630
result["db_name"] = string(*obj.DbName)
16241631
}
16251632

1633+
if obj.DbUniqueName != nil {
1634+
result["db_unique_name"] = string(*obj.DbUniqueName)
1635+
}
1636+
16261637
result["db_workload"] = string(obj.DbWorkload)
16271638

16281639
if obj.DefinedTags != nil {

website/docs/r/database_db_system.html.markdown

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,6 +79,7 @@ resource "oci_database_db_system" "test_db_system" {
7979
8080
#Optional
8181
database_software_image_id = oci_database_database_software_image.test_database_software_image.id
82+
db_unique_name = var.db_unique_name
8283
db_version = var.db_system_db_home_db_version
8384
defined_tags = var.db_system_db_home_defined_tags
8485
display_name = var.db_system_db_home_display_name
@@ -225,6 +226,7 @@ The following arguments are supported:
225226
* `time_stamp_for_point_in_time_recovery` - (Applicable when source=DATABASE) The point in time of the original database from which the new database is created. If not specifed, the latest backup is used to create the database.
226227
* `vault_id` - (Applicable when source=NONE) The [OCID](https://docs.cloud.oracle.com/iaas/Content/General/Concepts/identifiers.htm) of the Oracle Cloud Infrastructure [vault](https://docs.cloud.oracle.com/iaas/Content/KeyManagement/Concepts/keyoverview.htm#concepts).
227228
* `database_software_image_id` - (Applicable when source=DB_BACKUP | NONE) The database software image [OCID](https://docs.cloud.oracle.com/iaas/Content/General/Concepts/identifiers.htm) of the image to be used to restore a database.
229+
* `db_unique_name` - (Optional) The `DB_UNIQUE_NAME` of the Oracle Database.
228230
* `db_version` - (Required when source=NONE) A valid Oracle Database version. For a list of supported versions, use the ListDbVersions operation.
229231

230232
This cannot be updated in parallel with any of the following: licenseModel, dbEdition, cpuCoreCount, computeCount, computeModel, adminPassword, whitelistedIps, isMTLSConnectionRequired, openMode, permissionLevel, dbWorkload, privateEndpointLabel, nsgIds, isRefreshable, dbName, scheduledOperations, dbToolsDetails, isLocalDataGuardEnabled, or isFreeTier.

0 commit comments

Comments
 (0)