Merge d2640ac into 57c9075

.ci/magic-modules/generate-terraform.sh

@@ -29,6 +29,7 @@ bundle exec compiler -p products/redis -e terraform -o "${GOPATH}/src/github.com
 
 # Resources that were already using beta APIs before they started being autogenerated
 bundle exec compiler -v beta -p products/compute -t Address -e terraform -o "${GOPATH}/src/github.com/terraform-providers/terraform-provider-google/"
+bundle exec compiler -v beta -p products/compute -t Subnetwork -e terraform -o "${GOPATH}/src/github.com/terraform-providers/terraform-provider-google/"
 
 # This command can crash - if that happens, the script should not fail.
 set +e

products/compute/api.yaml

@@ -2709,6 +2709,7 @@ objects:
     name: 'Subnetwork'
     kind: 'compute#subnetwork'
     base_url: projects/{{project}}/regions/{{region}}/subnetworks
+    input: true
     exports:
       - !ruby/object:Api::Type::SelfLink
         name: 'selfLink'
@@ -2755,15 +2756,12 @@ objects:
           An optional description of this resource. Provide this property when
           you create the resource. This field can be set only at resource
           creation time.
-      # TODO(nelsonjr): Update 'gatewayAddress' input:true vs. output:true
-      # depending on the resolution of http://b/63113897.
       - !ruby/object:Api::Type::String
         name: 'gatewayAddress'
         description: |
           The gateway address for default routes to reach destination addresses
-          outside this subnetwork. This field can be set only at resource
-          creation time.
-        input: true
+          outside this subnetwork.
+        output: true
       - !ruby/object:Api::Type::Integer
         name: 'id'
         description: 'The unique identifier for the resource.'
@@ -2776,6 +2774,9 @@ objects:
           10.0.0.0/8 or 192.168.0.0/16. Ranges must be unique and
           non-overlapping within a network. Only IPv4 is supported.
         input: true
+        required: true
+        update_verb: :POST
+        update_url: 'projects/{{project}}/regions/{{region}}/subnetworks/{{name}}/expandIpCidrRange'
       - !ruby/object:Api::Type::String
         name: 'name'
         description: |
@@ -2794,11 +2795,54 @@ objects:
           The network this subnet belongs to.
           Only networks that are in the distributed mode can have subnetworks.
         input: true
+      - !ruby/object:Api::Type::Boolean
+        name: 'enableFlowLogs'
+        description: |
+          Whether to enable flow logging for this subnetwork.
+        min_version: beta
+      - !ruby/object:Api::Type::Fingerprint
+        name: 'fingerprint'
+        description: |
+          Fingerprint of this resource. A hash of the contents stored in this
+          object. This field is used in optimistic locking. This field will
+          be ignored when inserting a Subnetwork. An up-to-date fingerprint
+          must be provided in order to update the Subnetwork.
+        min_version: beta
+        update_verb: :PATCH
+        update_url:  projects/{{project}}/regions/{{region}}/subnetworks/{{name}}
+      - !ruby/object:Api::Type::Array
+        name: 'secondaryIpRanges'
+        description: |
+          An array of configurations for secondary IP ranges for VM instances
+          contained in this subnetwork. The primary IP of such VM must belong
+          to the primary ipCidrRange of the subnetwork. The alias IPs may belong
+          to either primary or secondary ranges.
+        min_version: beta
+        update_verb: :PATCH
+        update_url:  projects/{{project}}/regions/{{region}}/subnetworks/{{name}}
+        item_type: !ruby/object:Api::Type::NestedObject
+          properties:
+            - !ruby/object:Api::Type::String
+              name: 'rangeName'
+              description: |
+                The name associated with this subnetwork secondary range, used
+                when adding an alias IP range to a VM instance. The name must
+                be 1-63 characters long, and comply with RFC1035. The name
+                must be unique within the subnetwork.
+            - !ruby/object:Api::Type::String
+              name: 'ipCidrRange'
+              description: |
+                The range of IP addresses belonging to this subnetwork secondary
+                range. Provide this property when you create the subnetwork.
+                Ranges must be unique and non-overlapping with all primary and
+                secondary IP ranges within a network. Only IPv4 is supported.
       - !ruby/object:Api::Type::Boolean
         name: 'privateIpGoogleAccess'
         description: |
           Whether the VMs in this subnet can access Google services without
           assigned external IP addresses.
+        update_verb: :POST
+        update_url: 'projects/{{project}}/regions/{{region}}/subnetworks/{{name}}/setPrivateIpGoogleAccess'
       - !ruby/object:Api::Type::ResourceRef
         name: 'region'
         resource: 'Region'
@@ -2808,6 +2852,14 @@ objects:
           This field is not applicable to global addresses.
         required: true
         input: true
+    references: !ruby/object:Api::Resource::ReferenceLinks
+      guides:
+        'Private Google Access':
+          'https://cloud.google.com/vpc/docs/configure-private-google-access'
+        'Cloud Networking':
+          'https://cloud.google.com/vpc/docs/using-vpc'
+      api: 'https://cloud.google.com/compute/docs/reference/rest/beta/subnetworks'
+
   - !ruby/object:Api::Resource
     name: 'TargetHttpProxy'
     kind: 'compute#targetHttpProxy'

products/compute/terraform.yaml

@@ -469,13 +469,37 @@ overrides: !ruby/object:Provider::ResourceOverrides
         exclude: true
   Subnetwork: !ruby/object:Provider::Terraform::ResourceOverride
     id_format: "{{region}}/{{name}}"
-    exclude: true
     properties:
       id: !ruby/object:Provider::Terraform::PropertyOverride
         exclude: true
+      secondaryIpRanges: !ruby/object:Provider::Terraform::PropertyOverride
+        name: secondaryIpRange
+      fingerprint: !ruby/object:Provider::Terraform::PropertyOverride
+        exclude: false
       ipCidrRange: !ruby/object:Provider::Terraform::PropertyOverride
         validation: !ruby/object:Provider::Terraform::Validation
           function: 'validateIpCidrRange'
+    custom_code: !ruby/object:Provider::Terraform::CustomCode
+      constants: templates/terraform/constants/subnetwork.erb
+      resource_definition: templates/terraform/resource_definition/subnetwork.erb
+    examples: |
+      ```hcl
+        resource "google_compute_network" "custom-test" {
+          name = "%s"
+          auto_create_subnetworks = false
+        }
+
+        resource "google_compute_subnetwork" "network-with-private-secondary-ip-ranges" {
+          name = "%s"
+          ip_cidr_range = "10.2.0.0/16"
+          region = "us-central1"
+          network = "${google_compute_network.custom-test.self_link}"
+          secondary_ip_range {
+            range_name = "tf-test-secondary-range-update1"
+            ip_cidr_range = "192.168.10.0/24"
+          }
+        }
+      ```
   TargetHttpProxy: !ruby/object:Provider::Terraform::ResourceOverride
     examples: |
       ```hcl

provider/resource_overrides.rb

@@ -107,7 +107,13 @@ def override_objects
     end
 
     def override_properties(api_object, override)
-      override.properties.each do |property_path, property_override|
+      # We apply property overrides in reverse order of level of nesting.
+      # This helps us avoid a problem where we change the name of a property
+      # before we have applied the overrides to its child properties (and
+      # therefore can no longer find the child property, since the
+      # parent name has changed)
+      sorted_props = override.properties.sort_by { |path, _| -path.count('.') }
+      sorted_props.each do |property_path, property_override|
         check_property_value "properties['#{property_path}']",
                              property_override, Provider::PropertyOverride
         api_property = find_property api_object, property_path.split('.')

provider/terraform.rb

@@ -62,7 +62,9 @@ def updatable?(resource, properties)
 
     def force_new?(property, resource)
       !property.output &&
-        (property.input || (resource.input && property.update_url.nil?))
+        (property.input || (resource.input && property.update_url.nil? &&
+                            (property.parent.nil? ||
+                             force_new?(property.parent, resource))))
     end
 
     # Puts together the links to use to make API calls for a given resource type

provider/terraform/import.rb

@@ -28,6 +28,7 @@ module Import
       # c) short id w/o defaults: {{name}}
       #
       # Fields with default values are `project`, `region` and `zone`.
+      # rubocop:disable Metrics/AbcSize
       def import_id_formats(resource)
         if resource.import_format.nil? || resource.import_format.empty?
           underscored_base_url = resource.base_url.gsub(
@@ -40,6 +41,10 @@ def import_id_formats(resource)
           id_formats = resource.import_format
         end
 
+        unless resource.id_format.nil? || resource.id_format == '{{name}}'
+          id_formats << resource.id_format
+        end
+
         # short id: {{project}}/{{zone}}/{{name}}
         field_markers = id_formats[0].scan(/{{[[:word:]]+}}/)
         short_id_format = field_markers.join('/')
@@ -50,6 +55,7 @@ def import_id_formats(resource)
 
         (id_formats + [short_id_format, short_id_default_format]).uniq
       end
+      # rubocop:enable Metrics/AbcSize
     end
   end
 end

templates/terraform/constants/subnetwork.erb

@@ -0,0 +1,27 @@
+// Whether the IP CIDR change shrinks the block.
+func isShrinkageIpCidr(old, new, _ interface{}) bool {
+	_, oldCidr, oldErr := net.ParseCIDR(old.(string))
+	_, newCidr, newErr := net.ParseCIDR(new.(string))
+
+	if oldErr != nil || newErr != nil {
+		// This should never happen. The ValidateFunc on the field ensures it.
+		return false
+	}
+
+	oldStart, oldEnd := cidr.AddressRange(oldCidr)
+
+	if newCidr.Contains(oldStart) && newCidr.Contains(oldEnd) {
+		// This is a CIDR range expansion, no need to ForceNew, we have an update method for it.
+		return false
+	}
+
+	return true
+}
+
+
+func splitSubnetID(id string) (region string, name string) {
+	parts := strings.Split(id, "/")
+	region = parts[0]
+	name = parts[1]
+	return
+}

templates/terraform/flatten_property_method.erb

@@ -31,6 +31,9 @@ func flatten<%= prefix -%><%= titlelize_property(property) -%>(v interface{}) in
   <% end -%>
   return []interface{}{transformed}
 <% elsif property.is_a?(Api::Type::Array) && property.item_type.is_a?(Api::Type::NestedObject) -%>
+  if v == nil {
+    return v
+  }
   l := v.([]interface{})
   transformed := make([]interface{}, 0, len(l))
   for _, raw := range l {

templates/terraform/resource_definition/subnetwork.erb

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