Skip to content

Conversation

Dhruvkumar-Microsoft
Copy link
Contributor

Purpose

This pull request introduces new workflow files for the v2 deployment pipeline, expands support for the dev-v3 branch in existing workflows, and updates deployment configuration to use new container registry and image naming conventions. The changes streamline the build, deployment, and cleanup processes, improve resource management, and add support for MCP Docker image builds.

Workflow and CI/CD Pipeline Improvements:

  • Added .github/workflows/deploy-v2.yml to implement a comprehensive deployment workflow for the macae-v2 branch, including quota checks, resource provisioning, deployment, e2e testing, resource cleanup, and notification steps.
  • Added .github/workflows/docker-build-and-push-v2.yml to build and optionally push Docker images for backend and frontend services for the macae-v2 branch.

Branch and Tag Support Enhancements:

  • Extended branch support in .github/workflows/deploy.yml and .github/workflows/docker-build-and-push.yml to include the dev-v3 branch, and updated image tag logic to use *_v3 suffixes for relevant branches. [1] [2] [3] [4] [5]

Docker Image Build Improvements:

  • Added MCP Docker image build and push steps to .github/workflows/docker-build-and-push.yml, ensuring MCP service images are included in the CI/CD pipeline.

Deployment Configuration Updates:

  • Updated container registry hostnames, image names, and tags in infra/main.bicep to use the new registry (biabcontainerreg.azurecr.io) and standardized image naming for backend, frontend, and MCP services.
  • Added an empty availabilityZones array to the Bastion Host module configuration in infra/main.bicep for improved resource deployment control.

Documentation:

  • Minor update to docs/docker_mcp_server_testing.md to reference only the main deployment guide for troubleshooting.

Does this introduce a breaking change?

  • Yes
  • No

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install
  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

- main
- dev-v3
- hotfix
- dev
Copy link
Collaborator

Choose a reason for hiding this comment

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

can we remove dev from here for now?

IMAGE_TAG="latest_v3"
elif [[ "${{ env.BRANCH_NAME }}" == "dev" ]]; then
IMAGE_TAG="dev"
IMAGE_TAG="dev_v3"
Copy link
Collaborator

Choose a reason for hiding this comment

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

from here also

push:
branches:
- main
- dev
Copy link
Collaborator

Choose a reason for hiding this comment

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

From here as well

@Roopan-Microsoft Roopan-Microsoft merged commit 70e6b66 into macae-v3-dev-v2-vip Sep 22, 2025
3 of 4 checks passed
Copy link

🎉 This PR is included in version 2.2.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@Dhruvkumar-Microsoft Dhruvkumar-Microsoft deleted the psl-pipelinechanges branch September 24, 2025 05:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants