feat: add output for ECS service security group ID

[johncblandii]

what

• Added the service_sg_id

why

• This allows other services or dependent components to use this value to build upon this resource

references

Summary by CodeRabbit

• New Features
  • Added a new Terraform output exposing the ECS service security group ID, extending available infrastructure configuration values and providing additional visibility into service networking components.

[coderabbitai]

Walkthrough

A new Terraform output named service_sg_id was added to src/outputs.tf. This output exposes the ECS service security group ID from the module and includes a descriptive label for clarity.

Changes

Cohort / File(s)	Summary
New Output Export src/outputs.tf	Added public output service_sg_id that exposes one(module.ecs_alb_service_task[*].service_security_group_id) with description "The ECS service security group ID"

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

• Single-file change with straightforward output addition
• No logic changes, control flow modifications, or dependencies introduced
• Standard Terraform output pattern with clear intent

Poem

🐰 A new output blooms so bright,
The service group ID takes flight!
One line added, clean and true,
Security shines through and through! ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The pull request title "feat: add output for ECS service security group ID" directly and accurately describes the main change in the changeset. The title clearly indicates that a new output is being added for the ECS service security group ID, which aligns perfectly with the actual modification to src/outputs.tf where the "service_sg_id" output was introduced. The title is concise, specific, and avoids vague terminology, making it clear to anyone reviewing the project history what this change accomplishes.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1213803 and 2b69835.

📒 Files selected for processing (1)

• src/outputs.tf (1 hunks)

🧰 Additional context used

📓 Path-based instructions (3)

src/@(main|variables|outputs|providers|versions|context).tf

📄 CodeRabbit inference engine (AGENTS.md)

Keep the Terraform component as the source of truth in src/: maintain main.tf, variables.tf, outputs.tf, providers.tf, versions.tf, and context.tf there

Files:

• src/outputs.tf

**/*.tf

📄 CodeRabbit inference engine (AGENTS.md)

**/*.tf: Format Terraform with terraform fmt -recursive and do not commit formatting violations
Run TFLint and fix reported issues before committing
Use 2-space indentation in Terraform files

Files:

• src/outputs.tf

src/**/*.tf

📄 CodeRabbit inference engine (AGENTS.md)

In Terraform, prefer lower_snake_case for variables and locals; keep resource and data source names descriptive and aligned with Cloud Posse null-label patterns

Files:

• src/outputs.tf

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: Summary

🔇 Additional comments (1)

src/outputs.tf (1)

81-84: Output approved; confirmed by existing module usage.

The module creates an ECS Service for a web app via ALB. The new output service_sg_id is confirmed valid—the codebase already references one(module.ecs_alb_service_task[*].service_security_group_id) in existing security group rule configurations. The output follows consistent patterns with other module outputs in the file and adheres to naming and formatting conventions.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[mergify]

Thanks @johncblandii for creating this pull request!

A maintainer will review your changes shortly. Please don't be discouraged if it takes a while.

While you wait, make sure to review our contributor guidelines.

Tip

Need help or want to ask for a PR review to be expedited?

Join us on Slack in the #pr-reviews channel.

[github-actions]

These changes were released in v2.3.1.