Add azd-service-name tag to function app parameters#15
Add azd-service-name tag to function app parameters#15vhvb1989 wants to merge 1 commit intoAzure-Samples:mainfrom
Conversation
Missing tag for azd to know where to deploy
There was a problem hiding this comment.
Pull request overview
This PR aims to ensure Azure Developer CLI (azd) can correctly identify the Function App service to deploy by adding an azd-service-name tag to the API Function App module parameters.
Changes:
- Update the
apimodule parameters ininfra/main.bicepto include theazd-service-name: apitag viaunion(...).
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| tags: union(tags, { | ||
| 'azd-service-name': 'api' | ||
| }) |
There was a problem hiding this comment.
infra/app/api.bicep already applies the azd-service-name tag to the Function App resource (tags: union(tags, { 'azd-service-name': serviceName })). Adding the same tag again here makes the source of truth unclear and can drift if serviceName is ever overridden. Consider either (a) reverting to tags: tags here and relying on api.bicep, or (b) passing an explicit serviceName param from main.bicep and letting api.bicep set the tag once.
| tags: union(tags, { | |
| 'azd-service-name': 'api' | |
| }) | |
| tags: tags |
|
Good idea, already solved in a deeper file. thank you! |
Missing tag for azd to know where to deploy