Merge pull request #931 from julienvey/desc_firewall

provider/gce: Add description in firewall resource

builtin/providers/google/resource_compute_firewall.go

@@ -26,6 +26,11 @@ func resourceComputeFirewall() *schema.Resource {
 				ForceNew: true,
 			},
 
+			"description": &schema.Schema{
+				Type:     schema.TypeString,
+				Optional: true,
+			},
+
 			"network": &schema.Schema{
 				Type:     schema.TypeString,
 				Required: true,
@@ -306,6 +311,7 @@ func resourceFirewall(
 	// Build the firewall parameter
 	return &compute.Firewall{
 		Name:         d.Get("name").(string),
+		Description:  d.Get("description").(string),
 		Network:      network.SelfLink,
 		Allowed:      allowed,
 		SourceRanges: sourceRanges,

builtin/providers/google/resource_compute_firewall_test.go

@@ -126,6 +126,7 @@ resource "google_compute_network" "foobar" {
 
 resource "google_compute_firewall" "foobar" {
 	name = "terraform-test"
+	description = "Resource created for Terraform acceptance testing"
 	network = "${google_compute_network.foobar.name}"
 	source_tags = ["foo"]
 
@@ -142,6 +143,7 @@ resource "google_compute_network" "foobar" {
 
 resource "google_compute_firewall" "foobar" {
 	name = "terraform-test"
+	description = "Resource created for Terraform acceptance testing"
 	network = "${google_compute_network.foobar.name}"
 	source_tags = ["foo"]
 

website/source/docs/providers/google/r/compute_firewall.html.markdown

@@ -37,6 +37,8 @@ The following arguments are supported:
 * `name` - (Required) A unique name for the resource, required by GCE.
     Changing this forces a new resource to be created.
 
+* `description` - (Optional) Textual description field.
+
 * `network` - (Required) The name of the network to attach this firewall to.
 
 * `allow` - (Required) Can be specified multiple times for each allow