Merge pull request #3569 from hashicorp/b-meta-diffs-should-bump-serial

core: state metadata difference should bump serial
hashicorp · Oct 20, 2015 · 68e0133 · 68e0133
2 parents 15a36d0 + fca44bd
commit 68e0133
Show file tree

Hide file tree

Showing 2 changed files with 87 additions and 0 deletions.
diff --git a/terraform/state.go b/terraform/state.go
@@ -965,6 +965,21 @@ func (s *InstanceState) Equal(other *InstanceState) bool {
 		}
 	}
 
+	// Meta must be equal
+	if len(s.Meta) != len(other.Meta) {
+		return false
+	}
+	for k, v := range s.Meta {
+		otherV, ok := other.Meta[k]
+		if !ok {
+			return false
+		}
+
+		if v != otherV {
+			return false
+		}
+	}
+
 	return true
 }
 

diff --git a/terraform/state_test.go b/terraform/state_test.go
@@ -188,6 +188,43 @@ func TestStateEqual(t *testing.T) {
 				},
 			},
 		},
+
+		// Meta differs
+		{
+			false,
+			&State{
+				Modules: []*ModuleState{
+					&ModuleState{
+						Path: rootModulePath,
+						Resources: map[string]*ResourceState{
+							"test_instance.foo": &ResourceState{
+								Primary: &InstanceState{
+									Meta: map[string]string{
+										"schema_version": "1",
+									},
+								},
+							},
+						},
+					},
+				},
+			},
+			&State{
+				Modules: []*ModuleState{
+					&ModuleState{
+						Path: rootModulePath,
+						Resources: map[string]*ResourceState{
+							"test_instance.foo": &ResourceState{
+								Primary: &InstanceState{
+									Meta: map[string]string{
+										"schema_version": "2",
+									},
+								},
+							},
+						},
+					},
+				},
+			},
+		},
 	}
 
 	for i, tc := range cases {
@@ -224,6 +261,41 @@ func TestStateIncrementSerialMaybe(t *testing.T) {
 			},
 			1,
 		},
+		"S2 is different, but only via Instance Metadata": {
+			&State{
+				Serial: 3,
+				Modules: []*ModuleState{
+					&ModuleState{
+						Path: rootModulePath,
+						Resources: map[string]*ResourceState{
+							"test_instance.foo": &ResourceState{
+								Primary: &InstanceState{
+									Meta: map[string]string{},
+								},
+							},
+						},
+					},
+				},
+			},
+			&State{
+				Serial: 3,
+				Modules: []*ModuleState{
+					&ModuleState{
+						Path: rootModulePath,
+						Resources: map[string]*ResourceState{
+							"test_instance.foo": &ResourceState{
+								Primary: &InstanceState{
+									Meta: map[string]string{
+										"schema_version": "1",
+									},
+								},
+							},
+						},
+					},
+				},
+			},
+			4,
+		},
 		"S1 serial is higher": {
 			&State{Serial: 5},
 			&State{