Skip to content

Commit 3dcd660

Browse files
fix: address Copilot review comments - deploy.sh Dockerfile path and docs region example
- Fix deploy.sh: use backend/ApiApp.Dockerfile instead of WebApp.Dockerfile - Fix AZD_DEPLOYMENT.md: use eastus2 (allowed region) instead of eastus in example Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 8d1866b commit 3dcd660

3 files changed

Lines changed: 5 additions & 5 deletions

File tree

azure.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@ environment:
33
location: eastus
44

55
name: content-generation
6-
metadata:
7-
template: content-generation@1.22
6+
# metadata:
7+
# template: content-generation@1.22
88

99
requiredVersions:
1010
azd: '>= 1.18.0 != 1.23.9'

docs/AZD_DEPLOYMENT.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -227,7 +227,7 @@ Error: The model 'gpt-4o' is not available in region 'westeurope'
227227

228228
**Solution**: Set a different region for AI Services:
229229
```bash
230-
azd env set AZURE_ENV_AI_SERVICE_LOCATION eastus
230+
azd env set AZURE_ENV_AI_SERVICE_LOCATION eastus2
231231
```
232232

233233
#### 3. Container Build Fails

scripts/deploy.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -103,8 +103,8 @@ if [[ $REPLY =~ ^[Yy]$ ]]; then
103103
az acr build \
104104
--registry "$AZURE_ENV_CONTAINER_REGISTRY_NAME" \
105105
--image "contentgen-backend:$AZURE_ENV_IMAGE_TAG" \
106-
--file WebApp.Dockerfile \
107-
.
106+
--file backend/ApiApp.Dockerfile \
107+
backend
108108

109109
echo "✓ Container built and pushed to $AZURE_ENV_CONTAINER_REGISTRY_NAME.azurecr.io/contentgen-backend:$AZURE_ENV_IMAGE_TAG"
110110

0 commit comments

Comments
 (0)