Skip to content

Conversation

@Roopan-Microsoft
Copy link
Contributor

@Roopan-Microsoft Roopan-Microsoft commented Aug 18, 2025

Purpose

  • This pull request introduces significant improvements and fixes to the CI/CD pipeline, deployment process, and supporting scripts for the project. The main focus is on making the deployment pipeline more robust, automating the extraction and propagation of deployment URLs, disabling authentication for test environments, and enhancing Azure Search index creation for integrated vectorization. Additionally, the test automation workflow is refactored to support workflow calls, and several quality-of-life improvements are made to the Makefile and notification handling.

CI/CD Pipeline and Deployment Automation:

  • Major refactor of .github/workflows/ci.yml:

    • Renamed the main job to deploy, added outputs for deployment URLs and status, and introduced a workflow_dispatch trigger for manual runs. [1] [2]
    • Enhanced deployment steps: now captures deployment logs, extracts web/admin URLs via multiple methods, disables authentication, exports PostgreSQL host, and populates the database automatically.
    • Added a cleanup job to destroy resources after tests, and improved notification on failures. [1] [2]
    • Added an e2e-test job that runs only if deployment succeeds, using the extracted URLs.
  • Refactored .github/workflows/test-automation.yml to be reusable via workflow_call and accept deployment URLs as inputs, improving modularity and integration with the main pipeline.

Deployment and Utility Script Improvements:

  • Overhauled the Makefile deployment targets:
    • Improved login, provisioning, deployment, and extraction of frontend/admin URLs and PostgreSQL host for downstream use.
    • Added helper targets for checking authentication status and manually disabling authentication.

Azure Search Integration:

  • Enhanced Azure Search index creation in azure_search_helper.py:
    • Conditionally adds text and layoutText fields when integrated vectorization is enabled, supporting more advanced search scenarios.
    • Minor fix to AzureSearch credential handling for clarity.

Other Improvements:

  • Updated the coverage comment action in comment_coverage.yml to a newer commit for improved reliability.

CI/CD Pipeline and Deployment Automation

  • Refactored .github/workflows/ci.yml to automate deployment, URL extraction, authentication disabling, database population, and resource cleanup, with improved outputs and notifications. [1] [2] [3] [4] [5] [6]
  • Refactored .github/workflows/test-automation.yml to be triggered as a reusable workflow with URL inputs, enabling seamless end-to-end testing after deployment.

Deployment and Utility Script Improvements

  • Overhauled the Makefile deployment targets to robustly extract service URLs, PostgreSQL host, and provide helper targets for authentication and debugging.

Azure Search Integration

  • Added conditional fields for integrated vectorization in Azure Search index creation, and clarified credential handling in azure_search_helper.py. [1] [2]

Other Improvements

  • Updated the coverage comment GitHub Action to a newer version for improved reliability.

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

Roopan-Microsoft and others added 30 commits November 25, 2024 16:02
Co-authored-by: Roopan-Microsoft <[email protected]>
Co-authored-by: Ross Smith <[email protected]>
Co-authored-by: gpickett <[email protected]>
Co-authored-by: Francia Riesco <[email protected]>
Co-authored-by: Francia Riesco <[email protected]>
Co-authored-by: Prajwal D C <[email protected]>
…nd Update Conversation flow based on template selection (#1567)

Co-authored-by: Pavan Kumar <v-kupavan.microsoft.com>
Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Pavan-Microsoft <[email protected]>
Roopan-Microsoft and others added 13 commits June 16, 2025 18:35
…grated vectorization (#1839)

Co-authored-by: Vamshi-Microsoft <[email protected]>
Co-authored-by: Prajwal-Microsoft <[email protected]>
…test orchestration, and improved Makefile targets (#1855)

Co-authored-by: Harmanpreet Kaur <[email protected]>
…d maintenance (#1880)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Roopan P M <[email protected]>
Co-authored-by: Roopan-Microsoft <[email protected]>
Co-authored-by: Pavan-Microsoft <[email protected]>
Co-authored-by: Prasanjeet-Microsoft <[email protected]>
Co-authored-by: Harmanpreet Kaur <[email protected]>
@Roopan-Microsoft Roopan-Microsoft changed the title Dev fix: merging dev changes to main branch Aug 18, 2025
@Roopan-Microsoft Roopan-Microsoft added this pull request to the merge queue Aug 21, 2025
Merged via the queue into main with commit e77ae52 Aug 21, 2025
18 checks passed
@github-actions
Copy link

github-actions bot commented Oct 6, 2025

🎉 This PR is included in version 1.16.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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.