Skip to content
This repository was archived by the owner on May 27, 2025. It is now read-only.

Conversation

@jgbradley1
Copy link
Collaborator

This PR adds instructions and code changes to support the conversion of the existing deployment to an Azure Managed App for an improved deployment experience.

Manual deployments using the existing deployment guide and shell script (infra/deploy.sh) will always be supported as we seek to improve the deployment experience inside Azure.

@jgbradley1
Copy link
Collaborator Author

jgbradley1 commented Apr 2, 2025

Some additional changes included in this PR are

  • updated dependencies of the graphrag backend docker image
  • reverted the graphrag backend docker image to use a FastAPI app instead of using fastapi-offline. This was done to make the swagger docs page user friendly (users can now use the Try it out feature inside the swagger doc page to make REST calls. The attempt to use fastapi-offline led to other problems with APIM blocking the requests to the downloaded FastAPI static css files (making the entire swagger doc page unusable).
  • Reduced the number of deployment variables a user must define down to 2.
  • Reset the helm chart version=0.0.1 and defined appVersion=1.2.0. For future reference, we will start following the rule that appVersion will be a 1:1 mapping to the version of the graphrag library that is used in the accelerator for better clarity.

@jgbradley1 jgbradley1 self-assigned this Apr 2, 2025
@jgbradley1 jgbradley1 requested a review from a team April 2, 2025 14:37
stevetru1
stevetru1 previously approved these changes Apr 2, 2025
nievespg1
nievespg1 previously approved these changes Apr 2, 2025
@jgbradley1 jgbradley1 dismissed stale reviews from nievespg1 and stevetru1 via 711d913 April 2, 2025 15:29
@jgbradley1 jgbradley1 merged commit 5d2ab18 into main Apr 2, 2025
8 checks passed
@jgbradley1 jgbradley1 deleted the joshbradley/convert-to-managed-app branch April 2, 2025 21:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants