From 78a96f50bb3502f2af16f8fe8c21fd06f0bbc549 Mon Sep 17 00:00:00 2001 From: James Nugent Date: Mon, 22 Aug 2016 19:24:29 +0200 Subject: [PATCH] provider/google: Hook in state migration function As part of Terraform 0.7.1 it was observed in issue #8345 that the state migration for google_compute_firewall did not appear to be running, causing a panic when an uninitialized member was read. This commit hooks up the state migration function (which _was_ independently unit tested but was not actually in place). There is currently no good test framework for this, I will address this issue in a future RFC. --- builtin/providers/google/resource_compute_firewall.go | 1 + 1 file changed, 1 insertion(+) diff --git a/builtin/providers/google/resource_compute_firewall.go b/builtin/providers/google/resource_compute_firewall.go index b2da01c762..6cc8409de1 100644 --- a/builtin/providers/google/resource_compute_firewall.go +++ b/builtin/providers/google/resource_compute_firewall.go @@ -23,6 +23,7 @@ func resourceComputeFirewall() *schema.Resource { State: schema.ImportStatePassthrough, }, SchemaVersion: 1, + MigrateState: resourceComputeFirewallMigrateState, Schema: map[string]*schema.Schema{ "name": &schema.Schema{