Skip to content

Add CDN Endpoint Name for CDN Endpoint Module as output#194

Merged
SazniMohamed merged 1 commit intowso2:mainfrom
SazniMohamed:main-jan-5
Mar 16, 2026
Merged

Add CDN Endpoint Name for CDN Endpoint Module as output#194
SazniMohamed merged 1 commit intowso2:mainfrom
SazniMohamed:main-jan-5

Conversation

@SazniMohamed
Copy link
Contributor

@SazniMohamed SazniMohamed commented Mar 16, 2026

Purpose

Added a new "name" output to the CDN Front Door Endpoint module. This output exposes the endpoint name attribute, allowing users to directly reference this property in their Terraform configurations. This provides an additional way to access endpoint information alongside the existing host name and identifier outputs.

@coderabbitai
Copy link

coderabbitai bot commented Mar 16, 2026

Walkthrough

A new Terraform output named "name" is added to the CDN-FrontDoor-Endpoint module, exporting the endpoint's name attribute from the azurerm_cdn_frontdoor_endpoint resource. The existing id and host_name outputs remain unchanged.

Changes

Cohort / File(s) Summary
CDN FrontDoor Endpoint Output
modules/azurerm/CDN-FrontDoor-Endpoint/outputs.tf
Added new output "name" that exports the name attribute of the CDN FrontDoor endpoint resource, following the existing output pattern.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A name for the endpoint so bright,
New output declared just right,
The CDN speaks its identity clear,
A simple addition we all hold dear! 🚀

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description is incomplete. Only the Purpose section is filled with minimal detail; all other required template sections are missing or left as template prompts. Complete the PR description by filling out all required sections: Goals, Approach, User stories, Release note, Documentation, Training, Certification, Marketing, Automation tests (unit and integration), Security checks, Samples, Related PRs, Migrations, Test environment, and Learning. Provide concrete details rather than template placeholders.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: adding a 'name' output to the CDN Endpoint module.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
📝 Coding Plan
  • Generate coding plan for human review comments

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

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

Tip

You can customize the high-level summary generated by CodeRabbit.

Configure the reviews.high_level_summary_instructions setting to provide custom instructions for generating the high-level summary.

@SazniMohamed SazniMohamed merged commit 81a7895 into wso2:main Mar 16, 2026
5 checks passed
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