Cleaned up some TODOs.

modules/aws-network-existing/main.tf

@@ -21,3 +21,7 @@ data "aws_subnet" "subnets" {
   for_each = toset(data.aws_subnets.subnet-ids.ids)
   id = each.key
 }
+
+data "aws_security_group" "default" {
+  name = var.default-security-group-name
+}

modules/aws-network-existing/outputs.tf

@@ -2,12 +2,16 @@ output "default-vpc" {
   value = data.aws_vpc.default
 }
 
+output "default-sg" {
+  value = data.aws_security_group.default
+}
+
 output "subnets" {
   description = "An array of all subnets in default-vpc."
   value = data.aws_subnet.subnets
 }
 
-output "k8s-subnets" {
+output "k8s-subnets-ids" {
   description = "An array of subnets to be used for k8s VMs. These subnets were chosen by selecting a single subnet from each availability_zone."
   value = [for k,v in local.az-to-subnets : v[0]]
 }

modules/aws-network-existing/variables.tf

@@ -1,4 +1,7 @@
+variable "default-security-group-name" {
+  description = "The name of the existing default security group. This module will query AWS for a security group with this name,"
+}
+
 variable "default-vpc-name" {
   description = "The name of the existing default VPC. This module will query AWS for a VPC with this name,"
-  default = "Managed VPC"
 }

modules/aws-nodes/main.tf

@@ -1,8 +1,6 @@
 resource "aws_instance" "nodes" {
   ami                         = var.ami
   instance_type               = var.ec2-instance-type
-  # TODO REM double check this key.
-  # key_name                    = aws_key_pair.debug1.key_name
   # TODO Make this a variable.
   associate_public_ip_address = true
   subnet_id                   = var.subnet-id