Skip to content

azlocal to az#65

Merged
DrisDary merged 4 commits intomainfrom
sample-app-azlocal-to-az
Mar 23, 2026
Merged

azlocal to az#65
DrisDary merged 4 commits intomainfrom
sample-app-azlocal-to-az

Conversation

@DrisDary
Copy link
Copy Markdown
Contributor

@DrisDary DrisDary commented Mar 23, 2026

Motivation

Follow-up to #64 (Replace azlocal with az). Function App and Front Door, ACI and Blob Storage, and Web App and CosmosDB for NoSQL API has now az instead of azlocal. This PR ensures all sample apps consistently use az instead of azlocal.

Changes

  • ACI Blob Storage (samples/aci-blob-storage/python/): Removed unused AZ="azlocal" / AZ="az" branching blocks from terraform/deploy.sh, scripts/validate.sh, and scripts/cleanup.sh (the $AZ variable was defined but never referenced)
  • Web App CosmosDB NoSQL API (samples/web-app-cosmosdb-nosql-api/python/scripts/): Replaced all hardcoded azlocal CLI calls with az in deploy.sh and call-web-app.sh
  • Function App Front Door (samples/function-app-front-door/python/scripts/): Removed --use-localstack flag and azlocal interception lifecycle from deploy_all.sh and cleanup_all.sh. The script now auto-detects LocalStack via az account show --query environmentName and prints local endpoints accordingly
  • run-samples.sh: Removed --use-localstack flag from the front-door sample entry to match the updated script

Tests

  • Ran the Function App and Front Door sample locally against LocalStack — deployment completes successfully and local endpoints are printed correctly

Related

@DrisDary DrisDary self-assigned this Mar 23, 2026
@DrisDary DrisDary marked this pull request as ready for review March 23, 2026 14:18
@DrisDary DrisDary requested a review from a team as a code owner March 23, 2026 14:18
@DrisDary DrisDary marked this pull request as draft March 23, 2026 14:18
Copy link
Copy Markdown
Contributor

@paolosalvatori paolosalvatori left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks 🙏 Just make sure tests succeed, then feel free to merge updates.

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