Change default id format from {{name}} to self_link_uri (#2461)

* Change default id format from {{name}} to self_link_uri
GoogleCloudPlatform · Nov 13, 2019 · 99311c6 · 99311c6
1 parent aec98d6
commit 99311c6
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 3 deletions.
diff --git a/overrides/terraform/resource_override.rb b/overrides/terraform/resource_override.rb
@@ -76,7 +76,7 @@ def validate
         @examples ||= []
 
         check :legacy_name, type: String
-        check :id_format, type: String, default: '{{name}}'
+        check :id_format, type: String
         check :examples, item_type: Provider::Terraform::Examples, type: Array, default: []
         check :virtual_fields,
               item_type: Provider::Terraform::VirtualFields,

diff --git a/products/resourcemanager/terraform.yaml b/products/resourcemanager/terraform.yaml
@@ -18,6 +18,7 @@ overrides: !ruby/object:Overrides::ResourceOverrides
     autogen_async: true
     exclude: true
   Lien: !ruby/object:Overrides::Terraform::ResourceOverride
+    id_format: "{{name}}"
     import_format: ["{{parent}}/{{name}}"]
     examples:
       - !ruby/object:Provider::Terraform::Examples

diff --git a/provider/terraform.rb b/provider/terraform.rb
@@ -246,5 +246,12 @@ def build_object_data(object, output_folder, version)
     def extract_identifiers(url)
       url.scan(/\{\{(\w+)\}\}/).flatten
     end
+
+    # Returns the id format of an object, or self_link_uri if none is explicitly defined
+    # We prefer the long name of a resource as the id so that users can reference
+    # resources in a standard way, and most APIs accept short name, long name or self_link
+    def id_format(object)
+      object.id_format || object.self_link_uri
+    end
   end
 end
diff --git a/templates/terraform/resource.erb b/templates/terraform/resource.erb
@@ -191,7 +191,7 @@ func resource<%= resource_name -%>Create(d *schema.ResourceData, meta interface{
     }
 
     // Store the ID now
-    id, err := replaceVars(d, config, "<%= object.id_format -%>")
+    id, err := replaceVars(d, config, "<%= id_format(object) -%>")
     if err != nil {
         return fmt.Errorf("Error constructing id: %s", err)
     }
@@ -630,7 +630,7 @@ func resource<%= resource_name -%>Import(d *schema.ResourceData, meta interface{
     }
 
     // Replace import id for the resource id
-    id, err := replaceVars(d, config, "<%= object.id_format -%>")
+    id, err := replaceVars(d, config, "<%= id_format(object) -%>")
     if err != nil {
         return nil, fmt.Errorf("Error constructing id: %s", err)
     }