@@ -202,13 +202,18 @@ func resource_aws_instance_diff(
 
 		ComputedAttrs: []string{
 			"availability_zone",
+			"key_name",
 			"public_dns",
 			"public_ip",
 			"private_dns",
 			"private_ip",
+			"security_groups",
+			"subnet_id",
 		},
 	}
 
+	// TODO(mitchellh): figure out way to diff user_data_hash
+
 	return b.Diff(s, c)
 }
 
 
@@ -120,6 +120,13 @@ func (b *ResourceBuilder) Diff(
 					comp = true
 					break
 				}
+
+				// If the key is prefixed with the computed key, don't
+				// mark it for delete, ever.
+				if strings.HasPrefix(k, ck+".") {
+					comp = true
+					break
+				}
 			}
 			if comp {
 				continue
 
@@ -37,6 +37,34 @@ func TestResourceBuilder_attrSetComputed(t *testing.T) {
 	}
 }
 
+func TestResourceBuilder_attrSetComputedComplex(t *testing.T) {
+	rb := &ResourceBuilder{
+		Attrs: map[string]AttrType{
+			"foo": AttrTypeCreate,
+		},
+		ComputedAttrs: []string{
+			"foo",
+		},
+	}
+
+	state := &terraform.ResourceState{
+		ID: "foo",
+		Attributes: map[string]string{
+			"foo.#": "0",
+		},
+	}
+
+	c := testConfig(t, map[string]interface{}{}, nil)
+
+	diff, err := rb.Diff(state, c)
+	if err != nil {
+		t.Fatalf("err: %s", err)
+	}
+	if diff != nil {
+		t.Fatalf("diff shold be nil: %s", diff)
+	}
+}
+
 func TestResourceBuilder_replaceComputed(t *testing.T) {
 	rb := &ResourceBuilder{
 		Attrs: map[string]AttrType{
Original file line number	Diff line number	Diff line change
`@@ -120,6 +120,13 @@ func (b *ResourceBuilder) Diff(`
`120`	`120`	`comp = true`
`121`	`121`	`break`
`122`	`122`	`}`
	`123`	`+`
	`124`	`+ // If the key is prefixed with the computed key, don't`
	`125`	`+ // mark it for delete, ever.`
	`126`	`+ if strings.HasPrefix(k, ck+".") {`
	`127`	`+ comp = true`
	`128`	`+ break`
	`129`	`+ }`
`123`	`130`	`}`
`124`	`131`	`if comp {`
`125`	`132`	`continue`