-
Notifications
You must be signed in to change notification settings - Fork 13
Readme update to simplify azd deployment #9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
/assign @eamonoreilly |
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
Note this issue is getting in the way of clean checks and is independent of this PR: |
Signed-off-by: paulyuk <paulyuk@microsoft.com>
@eamonoreilly note a few issues/discrepancies with app settings made by Bicep vs. what's documented for AZD. Key names for things like CHAT_MODEL_DEPLOYMENT_NAME do not match. Also the value for OpenAiStorageConnection is set to a full connection string secret, and I think we want it instead to be a OpenAiStorageConnection__accountName. Would like to consult quickly on what we standardize on. |
Signed-off-by: paulyuk <paulyuk@microsoft.com>
Signed-off-by: paulyuk <paulyuk@microsoft.com>
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
… and Parameters completely Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
All manual tests pass. Readme updated. Ready to merge. @eamonoreilly |
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
Purpose
This is mostly a readme change to simplify the steps to deploy using AZD and reduce dependencies.
Additionally a small section is added to show how to run the apps locally (with remote AI resources).
Last, a few changes to the AZD bicep files were made to improve least permission grant in RBAC for each resource.
Does this introduce a breaking change?
Pull Request Type
What kind of change does this Pull Request introduce?
How to Test
What to Check
Verify that the following are valid