Skip to content

Commit 24d4a35

Browse files
authored
Merge pull request azavea#33 from azavea/feature/jrb/remove-iam-ec2-role
* Remove AmazonEC2ContainerServiceAutoscaleRole creation * Remove data resource lookup
2 parents 5ed24e4 + fa2e037 commit 24d4a35

File tree

3 files changed

+0
-20
lines changed

3 files changed

+0
-20
lines changed

README.md

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -90,8 +90,6 @@ module "container_service_cluster" {
9090
- `container_instance_security_group_id` - Security group ID of the EC2 container instances
9191
- `container_instance_ecs_for_ec2_service_role_name` - Name of IAM role associated with EC2 container instances
9292
- `ecs_service_role_name` - Name of IAM role for use with ECS services
93-
- `ecs_autoscale_role_name` - Name of IAM role for use with ECS service autoscaling
9493
- `ecs_service_role_arn` - ARN of IAM role for use with ECS services
95-
- `ecs_autoscale_role_arn` - ARN of IAM role for use with ECS service autoscaling
9694
- `container_instance_ecs_for_ec2_service_role_arn` - ARN of IAM role associated with EC2 container instances
9795
- `container_instance_autoscaling_group_name` - Name of container instance Autoscaling Group

main.tf

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -69,16 +69,6 @@ data "aws_iam_policy_document" "ecs_autoscale_assume_role" {
6969
}
7070
}
7171

72-
resource "aws_iam_role" "ecs_autoscale_role" {
73-
name = "ecs${title(var.environment)}AutoscaleRole"
74-
assume_role_policy = "${data.aws_iam_policy_document.ecs_autoscale_assume_role.json}"
75-
}
76-
77-
resource "aws_iam_role_policy_attachment" "ecs_service_autoscaling_role" {
78-
role = "${aws_iam_role.ecs_autoscale_role.name}"
79-
policy_arn = "arn:aws:iam::aws:policy/service-role/AmazonEC2ContainerServiceAutoscaleRole"
80-
}
81-
8272
#
8373
# Security group resources
8474
#

outputs.tf

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -18,10 +18,6 @@ output "ecs_service_role_name" {
1818
value = "${aws_iam_role.ecs_service_role.name}"
1919
}
2020

21-
output "ecs_autoscale_role_name" {
22-
value = "${aws_iam_role.ecs_autoscale_role.name}"
23-
}
24-
2521
output "container_instance_autoscaling_group_name" {
2622
value = "${aws_autoscaling_group.container_instance.name}"
2723
}
@@ -30,10 +26,6 @@ output "ecs_service_role_arn" {
3026
value = "${aws_iam_role.ecs_service_role.arn}"
3127
}
3228

33-
output "ecs_autoscale_role_arn" {
34-
value = "${aws_iam_role.ecs_autoscale_role.arn}"
35-
}
36-
3729
output "container_instance_ecs_for_ec2_service_role_arn" {
3830
value = "${aws_iam_role.container_instance_ec2.arn}"
3931
}

0 commit comments

Comments
 (0)