fixing the task_subnets logic

SuperCowPowers · Mar 3, 2025 · b443c37 · b443c37
1 parent 9debc20
commit b443c37
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/aws_setup/workbench_dashboard_full/workbench_dashboard_full/workbench_dashboard_stack.py b/aws_setup/workbench_dashboard_full/workbench_dashboard_full/workbench_dashboard_stack.py
@@ -122,7 +122,6 @@ def __init__(self, scope: Construct, id: str, props: WorkbenchDashboardStackProp
         container.add_port_mappings(ecs.PortMapping(container_port=8000))
 
         # Were we given a subnet selection?
-        # TODO: This isn't currently working, so we need to figure out how to use existing subnets
         subnet_selection = None
         if props.existing_subnet_ids:
             subnets = [
@@ -166,7 +165,7 @@ def __init__(self, scope: Construct, id: str, props: WorkbenchDashboardStackProp
             security_groups=[lb_security_group],
             open_listener=False,
             certificate=certificate,
-            # task_subnets=subnet_selection
+            task_subnets=subnet_selection
         )
 
         # Remove all default security groups from the load balancer