Skip to content

Commit

Permalink
revert the required part
Browse files Browse the repository at this point in the history
  • Loading branch information
catsby committed May 5, 2015
1 parent 1ed81da commit e9b08cf
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 7 deletions.
8 changes: 2 additions & 6 deletions builtin/providers/aws/resource_aws_security_group.go
Original file line number Diff line number Diff line change
Expand Up @@ -148,13 +148,9 @@ func resourceAwsSecurityGroupCreate(d *schema.ResourceData, meta interface{}) er

securityGroupOpts := &ec2.CreateSecurityGroupInput{}

if v, ok := d.GetOk("vpc_id"); ok {
if len(d.Get("egress").(*schema.Set).List()) == 0 {
return fmt.Errorf("Error creating Security Group: Security groups inside a VPC require an egress rule. See http://terraform.io/docs/providers/aws/r/security_group.html for more information.")
}

if v := d.Get("vpc_id"); v != nil {
securityGroupOpts.VPCID = aws.String(v.(string))
}
}

if v := d.Get("description"); v != nil {
securityGroupOpts.Description = aws.String(v.(string))
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ The following arguments are supported:
* `description` - (Required) The security group description.
* `ingress` - (Optional) Can be specified multiple times for each
ingress rule. Each ingress block supports fields documented below.
* `egress` - (Required, VPC only) Can be specified multiple times for each
* `egress` - (Optional, VPC only) Can be specified multiple times for each
egress rule. Each egress block supports fields documented below.
* `vpc_id` - (Optional) The VPC ID.
* `tags` - (Optional) A mapping of tags to assign to the resource.
Expand Down

0 comments on commit e9b08cf

Please sign in to comment.