Merge pull request #2 from silinternational/develop

removed availability_zones from aws/asg to resolve bug with terraform…

aws/asg/README.md

@@ -13,7 +13,6 @@ an auto scaling group that uses the configuration.
  - `app_env` - Name of environment, ex: production, testing, etc.
  - `ami_id` - ID for AMI to be used.
  - `aws_instance` - A map containing keys for `instance_type`, `volume_size`, `instance_count`
- - `aws_zones` - A list of availability zones to distribute instances across
  - `private_subnet_ids` - A list of private subnet ids to identify VPC subnets for placement
  - `default_sg_id` - VPC default security group ID to add instances to
  - `ecs_instance_profile_id` - IAM profile ID for ecsInstanceProfile
@@ -34,7 +33,6 @@ module "asg" {
   app_name = "${var.app_name}"
   app_env = "${var.app_env}"
   aws_instance = "${var.aws_instance}"
-  aws_zones = "${var.aws_zones}"
   private_subnet_ids = ["${module.vpc.private_subnet_ids}"]
   default_sg_id = "${module.vpc.vpc_default_sg_id}"
   ecs_instance_profile_id = "${module.ecs.ecs_instance_profile_id}"

aws/asg/main.tf

@@ -34,7 +34,6 @@ resource "aws_launch_configuration" "as_conf" {
  */
 resource "aws_autoscaling_group" "asg" {
   name                      = "asg-${var.app_name}-${var.app_env}-${aws_launch_configuration.as_conf.name}"
-  availability_zones        = "${var.aws_zones}"
   vpc_zone_identifier       = ["${var.private_subnet_ids}"]
   min_size                  = "${var.aws_instance["instance_count"]}"
   max_size                  = "${var.aws_instance["instance_count"]}"

aws/asg/vars.tf

@@ -25,16 +25,6 @@ variable "aws_instance" {
   }
 }
 
-variable "aws_zones" {
-  type = "list"
-
-  default = [
-    "us-east-1c",
-    "us-east-1d",
-    "us-east-1e",
-  ]
-}
-
 variable "private_subnet_ids" {
   type = "list"
 }