Skip to content

PLT-1625 Service Connect #437

Draft
mianava wants to merge 9 commits intomainfrom
mia/ecsservice/PLT-1625
Draft

PLT-1625 Service Connect #437
mianava wants to merge 9 commits intomainfrom
mia/ecsservice/PLT-1625

Conversation

@mianava
Copy link
Copy Markdown
Contributor

@mianava mianava commented Apr 9, 2026

🎫 Ticket

https://jira.cms.gov/browse/...

🛠 Changes

This PR extends the ECS service module to optionally support ECS Service Connect for mTLS between services, and ALB integration via an externally-provided HTTPS listener ARN. Both features are fully opt-in and should be non-breaking.

What is required change for the subscriber repo to upgrade to this version?

  1. dynamic "load_balancer" block now uses a single conditional block driven by local.enable_alb_integration rather than iterating over an external var.load_balancers list
  2. var.load_balancers (externally-managed TG attachments) has been removed — ALB integration is now fully managed by this module via alb_listener_arn
  3. service_name_override is retained; local.service_name resolves to var.service_name_override ?? var.platform.service, and local.service_name_full is used for all resource naming

How is ECS service connect configured?

  1. Added enable_ecs_service_connect flag (default: false)
  2. Added service_connect_namespace — Cloud Map namespace ARN, must be pre-associated with the ECS cluster
  3. Added service_connect_port_name — selects which named port mapping to expose via the Service
  4. Connect sidecar proxy; defaults to the first named port in port_mappings when only one is present
  5. Service Connect DNS name resolves to local.service_name within the namespace, enabling other services to call this one without hardcoded IPs or manual DNS construction

Some features were added to accommodate existing repository configurations and adoption of the common module.

ℹ️ Context

These changes are made to support mTLS

🧪 Validation

@mianava mianava marked this pull request as ready for review April 9, 2026 16:10
@mianava mianava requested a review from a team as a code owner April 9, 2026 16:10
@mianava mianava marked this pull request as draft April 10, 2026 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants