Skip to content

Conversation

@Uddipaan-Hazarika
Copy link
Contributor

@Uddipaan-Hazarika Uddipaan-Hazarika commented May 25, 2023

Context:

Terraform - VDB update call fails when appdata_source_params are absent.
https://delphix.atlassian.net/browse/HUBS-1815

Problem:

We were not specifying the field appdata_source_params as computed. And we were setting this field in the state file. So when we dont provide any value to the field(non-appdata), it was setting “null“ in the statefile. So any update call would change the string “null” → null. Which is not accepted as the param is a string. Hence, it failed.

resource_vdb_group_test.go:15: Step 1/1 error: After applying this test step, the plan was not empty.
        stdout:
        Terraform used the selected providers to generate the following execution
        plan. Resource actions are indicated with the following symbols:
          ~ update in-place
        
        Terraform will perform the following actions:
        
          # delphix_vdb.new will be updated in-place
          ~ resource "delphix_vdb" "new" {
              - appdata_source_params  = "null" -> null
                id                     = "11-ORACLE_DB_CONTAINER-8"
                name                   = "DBOMSRFB555Erhel8397k8qar8_MZ3"
                # (14 unchanged attributes hidden)
            }
        
        Plan: 0 to add, 1 to change, 0 to destroy.

Solution:

Added the computed flag. Now appdata_source_params doesn't gets set when we dont specify it. It is treated as a computed field if not set explicitly.

Testing

=== RUN   TestAccVdb_create_vdb_group_positive
[DELPHIX] [INFO] 2023/05/25 15:29:51 DCT-JobId:b46c1d07e50949199220cf614f59c810 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:29:57 DCT-JobId:b46c1d07e50949199220cf614f59c810 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:30:04 DCT-JobId:b46c1d07e50949199220cf614f59c810 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:30:10 DCT-JobId:b46c1d07e50949199220cf614f59c810 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:30:15 DCT-JobId:b46c1d07e50949199220cf614f59c810 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:30:21 DCT-JobId:b46c1d07e50949199220cf614f59c810 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:30:27 DCT-JobId:b46c1d07e50949199220cf614f59c810 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:30:32 DCT-JobId:b46c1d07e50949199220cf614f59c810 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:30:38 DCT-JobId:b46c1d07e50949199220cf614f59c810 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:30:44 DCT-JobId:b46c1d07e50949199220cf614f59c810 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:30:50 DCT-JobId:b46c1d07e50949199220cf614f59c810 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:30:56 DCT-JobId:b46c1d07e50949199220cf614f59c810 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:31:02 DCT-JobId:b46c1d07e50949199220cf614f59c810 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:31:08 DCT-JobId:b46c1d07e50949199220cf614f59c810 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:31:14 DCT-JobId:b46c1d07e50949199220cf614f59c810 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:31:20 DCT-JobId:b46c1d07e50949199220cf614f59c810 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:31:26 DCT-JobId:b46c1d07e50949199220cf614f59c810 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:31:32 DCT-JobId:b46c1d07e50949199220cf614f59c810 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:31:37 DCT-JobId:b46c1d07e50949199220cf614f59c810 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:31:43 DCT-JobId:b46c1d07e50949199220cf614f59c810 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:31:49 DCT-JobId:b46c1d07e50949199220cf614f59c810 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:31:54 DCT-JobId:b46c1d07e50949199220cf614f59c810 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:32:00 DCT-JobId:b46c1d07e50949199220cf614f59c810 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:32:06 DCT-JobId:b46c1d07e50949199220cf614f59c810 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:32:12 DCT-JobId:b46c1d07e50949199220cf614f59c810 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:32:18 DCT-JobId:b46c1d07e50949199220cf614f59c810 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:32:23 DCT-JobId:b46c1d07e50949199220cf614f59c810 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:32:29 DCT-JobId:b46c1d07e50949199220cf614f59c810 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:32:35 DCT-JobId:b46c1d07e50949199220cf614f59c810 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:32:41 DCT-JobId:b46c1d07e50949199220cf614f59c810 has Status:COMPLETED
[DELPHIX] [INFO] 2023/05/25 15:32:41 Job result is COMPLETED
[DELPHIX] [INFO] 2023/05/25 15:32:41 [OK] Breaking poll - Status 200 reached.
[DELPHIX] [INFO] 2023/05/25 15:32:42 VdbGroupId: db74a21cc30f400aaa83a3b85de1f802
[DELPHIX] [INFO] 2023/05/25 15:32:47 [OK] Breaking poll - Status 200 reached.
[DELPHIX] [INFO] 2023/05/25 15:32:47 VdbGroupId: db74a21cc30f400aaa83a3b85de1f802
[DELPHIX] [INFO] 2023/05/25 15:33:01 DCT-JobId:39690a9e48f746d585bdb6209c7dcc13 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:33:06 DCT-JobId:39690a9e48f746d585bdb6209c7dcc13 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:33:12 DCT-JobId:39690a9e48f746d585bdb6209c7dcc13 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:33:18 DCT-JobId:39690a9e48f746d585bdb6209c7dcc13 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:33:24 DCT-JobId:39690a9e48f746d585bdb6209c7dcc13 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:33:29 DCT-JobId:39690a9e48f746d585bdb6209c7dcc13 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:33:35 DCT-JobId:39690a9e48f746d585bdb6209c7dcc13 has Status:STARTED
[DELPHIX] [INFO] 2023/05/25 15:33:41 DCT-JobId:39690a9e48f746d585bdb6209c7dcc13 has Status:COMPLETED
[DELPHIX] [INFO] 2023/05/25 15:33:41 Job result is COMPLETED
[DELPHIX] [INFO] 2023/05/25 15:33:42 [OK] Breaking poll - Status 404 reached.
--- PASS: TestAccVdb_create_vdb_group_positive (246.66s)

@Uddipaan-Hazarika Uddipaan-Hazarika merged commit 42d6233 into delphix-integrations:develop May 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants