Skip to content

Add ora_db_unique_name Terraform variable to set DB_UNIQUE_NAME#436

Merged
mfielding merged 2 commits into
google:masterfrom
pythian:rel/oratk59-17
Jun 16, 2026
Merged

Add ora_db_unique_name Terraform variable to set DB_UNIQUE_NAME#436
mfielding merged 2 commits into
google:masterfrom
pythian:rel/oratk59-17

Conversation

@GMafra

@GMafra GMafra commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator

Change Description:

Add terraform support for setting the Oracle database unique name (DB_UNIQUE_NAME).
Pass the new --ora-db-unique-name flag from terraform through to install-oracle.sh.

This brings the terraform layer to parity with install-oracle.sh, which already
supports --ora-db-unique-name for both Primary and Standby databases.

Solution Overview:

Implements ora_db_unique_name support on the toolkit terraform configuration

Optional variable; works for both Primary and Standby databases. When left empty the
existing ansible default applies (db_name for Primary, db_name_s for Standby)

Test Commands:

terraform init
terraform apply -var-file=terraform.tfvars

Test Prep:
The following variable can be provided to terraform (optional)

ora_db_unique_name = "ora19p"

An example value was added to the terraform.tfvars.*.example files

Test 1: details:

Terraform renders the install-oracle.sh command with --ora-db-unique-name when the
variable is set, and omits it when left empty.

The implementation should be transparent for other components of toolkit

Expected Results:

When ora_db_unique_name is set, the generated install-oracle.sh command includes
--ora-db-unique-name <value>, and the database comes up with the expected
DB_UNIQUE_NAME for both Primary and Standby installs.

oratk59-17 terraform test results

database values

Add ora_db_unique_name Terraform variable to set DB_UNIQUE_NAME

@mfielding mfielding left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good; just on nit.

Comment thread terraform/terraform.tfvars.asm.example Outdated
@GMafra

GMafra commented Jun 16, 2026

Copy link
Copy Markdown
Collaborator Author

@mfielding nit addressed

@google-oss-prow

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: GMafra, mfielding

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@mfielding mfielding merged commit 3de554f into google:master Jun 16, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants