feat: Add deployment scripts and configuration for Azure Container Apps #447
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
…azure-setup-example
There was a problem hiding this comment.
Pull Request Overview
This PR introduces deployment scripts and configuration for running Outpost services on Azure Container Apps with Azure Service Bus as the message queue. The implementation provides both local Docker deployment and Azure Container Apps deployment options with comprehensive environment setup and validation capabilities.
Key changes:
- Azure dependencies provisioning with PostgreSQL, Redis, and Service Bus
- Local Docker Compose deployment configuration
- Azure Container Apps deployment automation
- Comprehensive diagnostics and validation tooling
Reviewed Changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| examples/azure/dependencies.sh | Provisions Azure infrastructure including PostgreSQL, Redis, Service Bus and service principal |
| examples/azure/local-deploy.sh | Deploys Outpost services locally using Docker Compose with Azure backends |
| examples/azure/azure-deploy.sh | Deploys Outpost services to Azure Container Apps |
| examples/azure/diagnostics.sh | Comprehensive validation and testing script for both local and Azure deployments |
| examples/azure/docker-compose.yml | Docker Compose configuration for local deployment |
| examples/azure/README.md | Complete deployment documentation and instructions |
| examples/azure/.gitignore | Excludes environment files from version control |
| --location "$LOCATION" \ | ||
| --sku Basic \ | ||
| --vm-size c0 | ||
| --enable-non-ssl-port # Comment out or remove if you only want SSL |
There was a problem hiding this comment.
Enabling non-SSL port for Redis poses a security risk. Consider using SSL-only connections or add proper documentation about the security implications.
| --enable-non-ssl-port # Comment out or remove if you only want SSL | |
| # SSL-only connections enforced for Redis |
| --location "$LOCATION" \ | ||
| --sku Basic \ | ||
| --vm-size c0 | ||
| --enable-non-ssl-port # Comment out or remove if you only want SSL |
There was a problem hiding this comment.
Missing line continuation backslash before the comment. This will cause the command to fail.
| --enable-non-ssl-port # Comment out or remove if you only want SSL | |
| --enable-non-ssl-port \ # Comment out or remove if you only want SSL |
|
|
||
| # Prepare environment variables for deployment | ||
| echo "Preparing environment variables for deployment..." | ||
| ENV_VARS_STRING=$(grep -v '^#' .env.runtime | sed 's/^export //g' | sed 's/"//g' | tr '\n' ' ') |
There was a problem hiding this comment.
The sed command removing quotes may break environment variables that contain spaces or special characters. Consider preserving quotes or using a more robust parsing method.
| ENV_VARS_STRING=$(grep -v '^#' .env.runtime | sed 's/^export //g' | sed 's/"//g' | tr '\n' ' ') | |
| ENV_VARS_STRING=$(awk -F= '/^[^#]/ {print $1 "=" $2}' .env.runtime | tr '\n' ' ') |
…nd component descriptions
Co-authored-by: Copilot <[email protected]>
…tgreSQL password handling
…ng for Azure deployment
Introduce scripts and configurations to deploy Outpost services on Azure Container Apps, including environment setup and validation steps.