Skip to content

Conversation

@Kanchan-Microsoft
Copy link
Contributor

Purpose

This pull request introduces a new, modular deployment and cleanup workflow for both Linux and Windows environments, enhancing automation, flexibility, and maintainability for Azure resource management and testing. It adds orchestrator and job workflows to centralize deployment logic, supports configurable options for resource cleanup and end-to-end testing, and updates code ownership.

Workflow orchestration and modularization:

  • Added deploy-orchestrator.yml to coordinate Docker build, deployment, testing, notification, and cleanup jobs, enabling reusable and maintainable workflow steps for deployments.
  • Introduced job-cleanup-deployment.yml to handle resource group cleanup with optimized Azure CLI commands and job summary reporting, improving resource management automation.

Deployment pipeline improvements:

  • Created new deployment workflows for Linux (deploy-linux.yml) and Windows (deploy-windows.yml), supporting flexible inputs for Azure location, resource group, WAF, Docker image build, cleanup, and test suite selection. These workflows invoke the orchestrator for streamlined execution.

Ownership and maintainability:

  • Updated .github/CODEOWNERS to add @dgp10801 as a default owner, ensuring proper code stewardship.

Does this introduce a breaking change?

  • Yes
  • No

Golden Path Validation

  • I have tested the primary workflows (the "golden path") to ensure they function correctly without errors.

Deployment Validation

  • I have validated the deployment process successfully and all services are running as expected with this change.

What to Check

Verify that the following are valid

  • I have built and tested the code locally and in a deployed app
  • For frontend changes, I have pulled the latest code from main, built the frontend, and committed all static files.
  • This is a change for all users of this app. No code or asset is specific to my use case or my organization.

Other Information

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.

4 participants