Azure / bicep-registry-modules

Bicep registry modules
MIT License
420 stars 277 forks source link

[AVM Module Issue]: ACA AVM should allow the creation of container apps without ingress #2205

Open mosabami opened 1 month ago

mosabami commented 1 month ago

Check for previous/existing GitHub issues

Issue Type?

Feature Request

Module Name

avm/res/app/container-app

(Optional) Module Version

No response

Description

One of the important reason for using container apps is to be able to easily create microservice applications that can communicate with themselves using its internal network. This allows customers to build applications where only a subset of the microservices can be accessed via ingress for security, cost and other reasons. The current AVM doesnt allow creation of those ingressless container-apps.

(Optional) Correlation Id

No response

microsoft-github-policy-service[bot] commented 1 month ago

[!IMPORTANT] The "Needs: Triage :mag:" label must be removed once the triage process is complete!

[!TIP] For additional guidance on how to triage this issue/PR, see the BRM Issue Triage documentation.

avm-team-linter[bot] commented 1 month ago

@mosabami, thanks for submitting this issue for the avm/res/app/container-app module!

[!IMPORTANT] A member of the @Azure/avm-res-app-containerapp-module-owners-bicep or @Azure/avm-res-app-containerapp-module-contributors-bicep team will review it soon!

oZakari commented 1 month ago

Thanks @mosabami, will work on getting this incorporated hopefully this month.