-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
CopilotChat: Replace "SK" with "SemanticKernel" and add App Service Plan allowable values. #1109
Conversation
Still doing some validation - hold merge until I've finished. |
This change is missing: Package URI from https://skaasdeploy.blob.core.windows.net/api/skaas.zip to be https://semantic-kernel-deploy.blob.core.windows.net/api/Semantic-Kernel.zip |
We'll need to standup another blob service and populate it. I wanted to not block these changes on that move. I'll look at that next. |
samples/apps/copilot-chat-app/webapi/DeploymentTemplates/sk-existing-azureopenai.bicep
Outdated
Show resolved
Hide resolved
samples/apps/copilot-chat-app/webapi/DeploymentTemplates/sk-existing-openai.bicep
Outdated
Show resolved
Hide resolved
samples/apps/copilot-chat-app/webapi/DeploymentTemplates/sk-new.bicep
Outdated
Show resolved
Hide resolved
samples/apps/copilot-chat-app/webapi/DeploymentTemplates/main.bicep
Outdated
Show resolved
Hide resolved
…bicep Co-authored-by: Gil LaHaye <gillahaye@microsoft.com>
…w.bicep Co-authored-by: Gil LaHaye <gillahaye@microsoft.com>
…isting-azureopenai.bicep Co-authored-by: Gil LaHaye <gillahaye@microsoft.com>
samples/apps/copilot-chat-app/webapi/DeploymentTemplates/main.bicep
Outdated
Show resolved
Hide resolved
…isting-openai.bicep Co-authored-by: Gil LaHaye <gillahaye@microsoft.com>
…bicep Co-authored-by: Gil LaHaye <gillahaye@microsoft.com>
…lan allowable values. (microsoft#1109) ### Motivation and Context Remove most occurrences of "SK" from CopilotChat deployment ### Description - Renamed most occurrences of "SK" to either "SemanticKernel" or "SemKer" throughout CopilotChat - Updated App Plan SKUs with a list of allowed values. --------- Co-authored-by: Harleen Thind <39630244+hathind-ms@users.noreply.github.com> Co-authored-by: Gil LaHaye <gillahaye@microsoft.com>
…lan allowable values. (microsoft#1109) ### Motivation and Context Remove most occurrences of "SK" from CopilotChat deployment ### Description - Renamed most occurrences of "SK" to either "SemanticKernel" or "SemKer" throughout CopilotChat - Updated App Plan SKUs with a list of allowed values. --------- Co-authored-by: Harleen Thind <39630244+hathind-ms@users.noreply.github.com> Co-authored-by: Gil LaHaye <gillahaye@microsoft.com>
Motivation and Context
Remove most occurrences of "SK" from CopilotChat deployment
Description