delphix-integrations / terraform-provider-delphix

Repository for Delphix Terraform Provider built on DCT APIs.
https://integrations.delphix.com
11 stars 9 forks source link

HUBS-1804 | Dependabot module updates and version bump #58

Closed Uddipaan-Hazarika closed 1 year ago

Uddipaan-Hazarika commented 1 year ago

Context:

Dependabot changes and version bump for latest update

Problem:

There were 3 open PR directly to main that are addressed here.

Solution:

bumped the version according to the latest dependabot change and provider version.

Testing:

After updating the version, ran a provision job for postgresql. TF: `resource "delphix_vdb" "test_postgres_tf" { auto_select_repository = true source_data_id = "10-APPDATA_CONTAINER-1" name = "test_postgres_tf3"

appdata_source_params = jsonencode({ mountLocation = "/mnt/GAT" postgresPort = 5434 configSettingsStg = [{ propertyName: "timezone", value:"GMT", commentProperty:false}] })

appdata_config_params = jsonencode({ name = "Postgres-5434 - /mnt/GAT/data2" })

config_params = jsonencode({ processes = 151 })

make_current_account_owner = false }`

Output: `Terraform used the selected providers to generate the following execution plan. Resource actions are indicated with the following symbols:

Terraform will perform the following actions:

delphix_vdb.test_postgres_tf will be created

Plan: 1 to add, 0 to change, 0 to destroy. delphix_vdb.test_postgres_tf: Creating... delphix_vdb.test_postgres_tf: Still creating... [10s elapsed] delphix_vdb.test_postgres_tf: Still creating... [20s elapsed] delphix_vdb.test_postgres_tf: Still creating... [30s elapsed] delphix_vdb.test_postgres_tf: Still creating... [40s elapsed] delphix_vdb.test_postgres_tf: Creation complete after 44s [id=10-APPDATA_CONTAINER-3]

Apply complete! Resources: 1 added, 0 changed, 0 destroyed.`

nick-mathison commented 1 year ago

other than the version bump. I'm not sure about the implications here. I will bow out.