Add enabled var (#11)

Author: Lethgir

README.md

@@ -9,6 +9,7 @@ Features:
 - Saving cost using a spot instance (from $1/month)
 - Fixed source IP address by reattaching ENI
 - Supporting Systems Manager Session Manager
+- Disable costly resources for compatibility with workspaces
 
 Terraform 0.12 is required.
 
@@ -153,6 +154,7 @@ This is an open source software. Feel free to open issues and pull requests.
 
 | Name | Description | Type | Default | Required |
 |------|-------------|:----:|:-----:|:-----:|
+| enabled | Enable or disable costly resources| boolean | `true` | no |
 | extra\_user\_data | Extra script to run in the NAT instance | string | `""` | no |
 | image\_id | AMI of the NAT instance. Default to the latest Amazon Linux 2 | string | `""` | no |
 | instance\_types | Candidates of instance type for the NAT instance. This is used in the mixed instances policy | list | `[ "t3.nano", "t3a.nano" ]` | no |
@@ -166,6 +168,8 @@ This is an open source software. Feel free to open issues and pull requests.
 
 ## Outputs
 
+Outputs are empty string for disabled costly resources.
+
 | Name | Description |
 |------|-------------|
 | eip\_id | ID of the Elastic IP |

main.tf

@@ -36,6 +36,7 @@ resource "aws_network_interface" "this" {
 }
 
 resource "aws_eip" "this" {
+  count             = var.enabled ? 1 : 0
   network_interface = aws_network_interface.this.id
   tags = {
     Name = "nat-instance-${var.name}"
@@ -105,8 +106,8 @@ resource "aws_launch_template" "this" {
 
 resource "aws_autoscaling_group" "this" {
   name_prefix         = var.name
-  desired_capacity    = 1
-  min_size            = 1
+  desired_capacity    = var.enabled ? 1 : 0
+  min_size            = var.enabled ? 1 : 0
   max_size            = 1
   vpc_zone_identifier = [var.public_subnet]
 

outputs.tf

@@ -1,30 +1,30 @@
 output "eip_id" {
   description = "ID of the Elastic IP"
-  value       = "${aws_eip.this.id}"
+  value       = var.enabled ? aws_eip.this[0].id : ""
 }
 
 output "eip_public_ip" {
   description = "Public IP of the Elastic IP for the NAT instance"
-  value       = "${aws_eip.this.public_ip}"
+  value       = var.enabled ? aws_eip.this[0].public_ip : ""
 }
 
 output "eni_id" {
   description = "ID of the ENI for the NAT instance"
-  value       = "${aws_network_interface.this.id}"
+  value       = aws_network_interface.this.id
 }
 
 output "eni_private_ip" {
   description = "Private IP of the ENI for the NAT instance"
   # workaround of https://github.com/terraform-providers/terraform-provider-aws/issues/7522
-  value = "${tolist(aws_network_interface.this.private_ips)[0]}"
+  value = tolist(aws_network_interface.this.private_ips)[0]
 }
 
 output "sg_id" {
   description = "ID of the security group of the NAT instance"
-  value       = "${aws_security_group.this.id}"
+  value       = aws_security_group.this.id
 }
 
 output "iam_role_name" {
   description = "Name of the IAM role for the NAT instance"
-  value       = "${aws_iam_role.this.name}"
+  value       = aws_iam_role.this.name
 }

variables.tf

@@ -1,3 +1,7 @@
+variable "enabled" {
+  description = "Enable or not costly resources"
+  default     = true
+}
 variable "name" {
   description = "Name for all the resources as identifier"
 }