Disable use of public subnets for EMR

wri · Apr 22, 2024 · 816e5c7 · 816e5c7
1 parent 8faa931
commit 816e5c7
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 8 deletions.
diff --git a/terraform/main.tf b/terraform/main.tf
@@ -54,7 +54,6 @@ module "datapump" {
   glad_path = local.glad_path
   emr_instance_profile_name = data.terraform_remote_state.core.outputs.emr_instance_profile_name
   emr_service_role_name = data.terraform_remote_state.core.outputs.emr_service_role_name
-  public_subnet_ids = data.terraform_remote_state.core.outputs.public_subnet_ids
   ec2_key_name = data.terraform_remote_state.core.outputs.key_pair_jterry_gfw
   gcs_secret_arn = data.terraform_remote_state.core.outputs.secrets_read-gfw-gee-export_arn
   read_gfw_api_secrets_policy = data.terraform_remote_state.core.outputs.secrets_read-gfw-api-token_policy_arn

diff --git a/terraform/modules/datapump/lambdas.tf b/terraform/modules/datapump/lambdas.tf
@@ -45,7 +45,6 @@ resource "aws_lambda_function" "executor" {
       S3_BUCKET_PIPELINE             = var.pipelines_bucket
       S3_BUCKET_DATA_LAKE            = var.data_lake_bucket
       GEOTRELLIS_JAR_PATH            = var.geotrellis_jar_path
-      PUBLIC_SUBNET_IDS              = jsonencode(var.public_subnet_ids)
       EC2_KEY_NAME                   = var.ec2_key_name
       EMR_SERVICE_ROLE               = var.emr_service_role_name
       EMR_INSTANCE_PROFILE           = var.emr_instance_profile_name

diff --git a/terraform/modules/datapump/variables.tf b/terraform/modules/datapump/variables.tf
@@ -65,12 +65,6 @@ variable "ec2_key_name" {
   description = "Key pair to use for SSHing into EC2"
 }
 
-variable "public_subnet_ids" {
-  default     = []
-  type        = list(string)
-  description = "Public subnet IDs to run on"
-}
-
 variable "pipelines_bucket" {
   type        = string
   description = "Pipelines bucket to store intermediate results"