Support unordered lists in Terraform.

products/compute/terraform.yaml

@@ -623,6 +623,7 @@ overrides: !ruby/object:Provider::ResourceOverrides
           function: 'validateGCPName'
       secondaryIpRanges: !ruby/object:Provider::Terraform::PropertyOverride
         name: secondaryIpRange
+        unordered_list: true
       secondaryIpRanges.rangeName: !ruby/object:Provider::Terraform::PropertyOverride
         validation: !ruby/object:Provider::Terraform::Validation
           function: 'validateGCPName'

provider/terraform/property_override.rb

@@ -24,6 +24,8 @@ module OverrideFields
       attr_reader :state_func # Adds a StateFunc to the schema
       attr_reader :sensitive # Adds `Sensitive: true` to the schema
       attr_reader :validation # Adds a ValidateFunc to the schema
+      # Indicates that this is an Array that should have Set diff semantics.
+      attr_reader :unordered_list
 
       attr_reader :is_set # Uses a Set instead of an Array
       # Optional function to determine the unique ID of an item in the set
@@ -92,6 +94,7 @@ def validate
         # Ensures boolean values are set to false if nil
         @sensitive ||= false
         @is_set ||= false
+        @unordered_list ||= false
         @default_from_api ||= false
 
         check_property :sensitive, :boolean

templates/terraform/resource.erb

@@ -36,6 +36,9 @@ func resource<%= resource_name -%>() *schema.Resource {
 		Update: resource<%= resource_name -%>Update,
 		<% end -%>
 		Delete: resource<%= resource_name -%>Delete,
+        <% if settable_properties.any? {|p| p.unordered_list} && !object.custom_code.resource_definition -%>
+        CustomizeDiff: resource<%= resource_name -%>CustomizeDiff,
+        <% end -%>
 
 		Importer: &schema.ResourceImporter{
 			State: resource<%= resource_name -%>Import,
@@ -74,6 +77,14 @@ func resource<%= resource_name -%>() *schema.Resource {
 		},
 	}
 }
+<% if settable_properties.any? {|p| p.unordered_list} -%>
+func resource<%= resource_name -%>CustomizeDiff(diff *schema.ResourceDiff, meta interface{}) error {
+<% settable_properties.select {|p| p.unordered_list}.each do |prop| -%>
+<%= compile_template('templates/terraform/unordered_list_customize_diff.erb',
+			         prop: prop) -%>
+<% end -%>
+}
+<% end -%>
 
 func resource<%= resource_name -%>Create(d *schema.ResourceData, meta interface{}) error {
 	config := meta.(*Config)

templates/terraform/resource_definition/subnetwork.erb

@@ -1,3 +1,4 @@
 CustomizeDiff: customdiff.All(
 		customdiff.ForceNewIfChange("ip_cidr_range", isShrinkageIpCidr),
+        resourceComputeSubnetworkCustomizeDiff,
 ),

templates/terraform/unordered_list_customize_diff.erb

@@ -0,0 +1,34 @@
+keys := diff.GetChangedKeysPrefix(<%= go_literal(Google::StringUtils.underscore(prop.name)) -%>)
+if len(keys) < 1 {
+	return nil
+}
+count := diff.Get("<%= Google::StringUtils.underscore(prop.name) -%>.#").(int)
+if count < 1 {
+	return nil
+}
+old := make([]interface{}, count)
+new := make([]interface{}, count)
+for i := 0; i < count; i++ {
+    o, n := diff.GetChange(fmt.Sprintf("<%= Google::StringUtils.underscore(prop.name) -%>.%d", i))
+
+    if o != nil {
+        old = append(old, o)
+    }
+    if n != nil {
+        new = append(new, n)
+    }
+}
+if len(old) != len(new) {
+    return nil
+}
+
+for o, _ := range old {
+    for n, _ := range new {
+        if reflect.DeepEqual(o, n) {
+            break
+        }
+        return nil
+    }
+}
+
+return diff.Clear(<%= go_literal(Google::StringUtils.underscore(prop.name)) -%>)