Skip to content

Conversation

Dhruvkumar-Microsoft
Copy link
Contributor

Purpose

This pull request updates the deployment documentation and infrastructure to add Azure Storage and Azure Search resources, and ensures that the backend is configured with the necessary environment variables for these new services. The documentation has also been improved for clarity around installing requirements and deploying local changes.

Infrastructure enhancements:

  • Added Azure Storage Account and Azure Search Service modules to main_custom.bicep, including networking, private endpoints, and role assignments. This provides the application with blob storage and search capabilities.
  • Exported new outputs for storage and search details (e.g., blob URL, storage account/container name, search endpoint/name) from main_custom.bicep for use in the application and deployment pipelines.
  • Updated backend container app environment variables to include the new storage and search configuration, ensuring the backend has access to these services at runtime.

Documentation improvements:

  • Clarified and separated the installation steps for backend and frontend Python requirements in DeploymentGuide.md, specifying the use of uv for backend and pip for frontend.
  • Added instructions for activating the backend virtual environment before running the backend server, and updated step numbers for running and accessing the application. [1] [2]
  • Provided new guidance for deploying local changes, including renaming configuration files and updating deployment steps.

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

@Fr4nc3
Copy link
Collaborator

Fr4nc3 commented Sep 3, 2025

Hi [Dhruvkumar-Microsoft] we don't need fastmcp on the backend

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