From b32a863ed3d643729f00b0896b54fb2599e5f997 Mon Sep 17 00:00:00 2001 From: James Nugent Date: Thu, 18 Feb 2016 08:37:25 -0800 Subject: [PATCH] provider/google: Clarify comment about defaults --- builtin/providers/google/resource_compute_network.go | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/builtin/providers/google/resource_compute_network.go b/builtin/providers/google/resource_compute_network.go index 509ddf11a..d57290103 100644 --- a/builtin/providers/google/resource_compute_network.go +++ b/builtin/providers/google/resource_compute_network.go @@ -38,8 +38,11 @@ func resourceComputeNetwork() *schema.Resource { Type: schema.TypeBool, Optional: true, ForceNew: true, - Default: false, // TODO: ideally should be true to match Google's default behaviour, but this causes backward - // compatibility issue with existing terraform configs + /* Ideally this would default to true as per the API, but that would cause + existing Terraform configs which have not been updated to report this as + a change. Perhaps we can bump this for a minor release bump rather than + a point release. */ + Default: false, }, "description": &schema.Schema{