From ff0741a12effa92695eb45f0563494941772718f Mon Sep 17 00:00:00 2001
From: Jack Pearkes <jackpearkes@gmail.com>
Date: Tue, 15 Jul 2014 12:40:20 -0400
Subject: [PATCH] providers/aws: security group validations

---
 builtin/providers/aws/resource_aws_security_group.go | 12 ++++++++++++
 builtin/providers/aws/resources.go                   |  9 +++++----
 2 files changed, 17 insertions(+), 4 deletions(-)

diff --git a/builtin/providers/aws/resource_aws_security_group.go b/builtin/providers/aws/resource_aws_security_group.go
index 050d68f47..f6bf85035 100644
--- a/builtin/providers/aws/resource_aws_security_group.go
+++ b/builtin/providers/aws/resource_aws_security_group.go
@@ -5,6 +5,7 @@ import (
 	"log"
 
 	"github.com/hashicorp/terraform/flatmap"
+	"github.com/hashicorp/terraform/helper/config"
 	"github.com/hashicorp/terraform/helper/diff"
 	"github.com/hashicorp/terraform/terraform"
 	"github.com/mitchellh/goamz/ec2"
@@ -177,3 +178,14 @@ func resource_aws_security_group_retrieve(id string, ec2conn *ec2.EC2) (*ec2.Sec
 
 	return &sg, nil
 }
+
+func resource_aws_security_group_validation() *config.Validator {
+	return &config.Validator{
+		Required: []string{
+			"name",
+		},
+		Optional: []string{
+			"description",
+		},
+	}
+}
diff --git a/builtin/providers/aws/resources.go b/builtin/providers/aws/resources.go
index 55f21f679..b3f4181a9 100644
--- a/builtin/providers/aws/resources.go
+++ b/builtin/providers/aws/resources.go
@@ -93,10 +93,11 @@ func init() {
 			},
 
 			"aws_security_group": resource.Resource{
-				Create:  resource_aws_security_group_create,
-				Destroy: resource_aws_security_group_destroy,
-				Diff:    resource_aws_security_group_diff,
-				Refresh: resource_aws_security_group_refresh,
+				ConfigValidator: resource_aws_security_group_validation(),
+				Create:          resource_aws_security_group_create,
+				Destroy:         resource_aws_security_group_destroy,
+				Diff:            resource_aws_security_group_diff,
+				Refresh:         resource_aws_security_group_refresh,
 			},
 
 			"aws_subnet": resource.Resource{