Fix hasComputedValue to handle nested slices

pkg/provider/util.go

@@ -31,6 +31,19 @@ func hasComputedValue(obj *unstructured.Unstructured) bool {
 			if field, isMap := v.(map[string]interface{}); isMap {
 				objects = append(objects, field)
 			}
+			if field, isSlice := v.([]interface{}); isSlice {
+				for _, v := range field {
+					if _, isComputed := v.(resource.Computed); isComputed {
+						return true
+					}
+					if field, isMap := v.(map[string]interface{}); isMap {
+						objects = append(objects, field)
+					}
+				}
+			}
+			if field, isSlice := v.([]map[string]interface{}); isSlice {
+				objects = append(objects, field...)
+			}
 		}
 	}
 

pkg/provider/util_test.go

@@ -92,6 +92,26 @@ func TestHasComputedValue(t *testing.T) {
 			}},
 			hasComputedValue: true,
 		},
+		{
+			name: "Object with nested slice of map[string]interface{} has a computed value",
+			obj: &unstructured.Unstructured{Object: map[string]interface{}{
+				"field1": 1,
+				"field2": []map[string]interface{}{
+					{"field3": resource.Computed{}},
+				},
+			}},
+			hasComputedValue: true,
+		},
+		{
+			name: "Object with nested slice of interface{} has a computed value",
+			obj: &unstructured.Unstructured{Object: map[string]interface{}{
+				"field1": 1,
+				"field2": []interface{}{
+					resource.Computed{},
+				},
+			}},
+			hasComputedValue: true,
+		},
 	}
 
 	for _, test := range tests {