Skip to content
This repository has been archived by the owner on Jun 30, 2022. It is now read-only.

[Docs] Confusing qnaEndpoint semantics in deploy.ps1 script #3544

Closed
GitDruid opened this issue Jul 8, 2020 · 4 comments · Fixed by #3557
Closed

[Docs] Confusing qnaEndpoint semantics in deploy.ps1 script #3544

GitDruid opened this issue Jul 8, 2020 · 4 comments · Fixed by #3557
Assignees
Labels
Bot Services Required for internal Azure reporting. Do not delete. Do not change color. customer-replied-to Indicates that the team has replied to the issue reported by the customer. Do not delete. customer-reported Issue is created by anyone that is not a collaborator in the repository. Type: Bug Something isn't working

Comments

@GitDruid
Copy link

GitDruid commented Jul 8, 2020

Please describe your request

Provisioning of the QnA resource does not set the region based on the $location parameter. West US Endpoint is hardcoded:
$qnaEndpoint = "https://westus.api.cognitive.microsoft.com/qnamaker/v4.0"
So, if you don't provide it, you will not be able to create the resource in westeurope, for instance.

Is this an issue with current documentation? If yes, please provide the link.

Current documentation here:
https://microsoft.github.io/botframework-solutions/virtual-assistant/handbook/deployment-scripts/
reports that endpoint for deploying QnA Maker knowledge bases is only required for Azure Gov deployments.

@xieofxie xieofxie transferred this issue from microsoft/botframework-components Jul 9, 2020
@ryanisgrig ryanisgrig self-assigned this Jul 9, 2020
@ryanisgrig
Copy link
Collaborator

ryanisgrig commented Jul 9, 2020

Thanks for the feedback! We will follow up with this after investigating.

@ryanisgrig ryanisgrig added customer-replied-to Indicates that the team has replied to the issue reported by the customer. Do not delete. customer-reported Issue is created by anyone that is not a collaborator in the repository. Status: Backlog This will go in the product backlog Type: Bug Something isn't working labels Jul 10, 2020
@lauren-mills
Copy link
Contributor

Hi @GitDruid, when QnA Maker is deployed, it creates three resources in Azure, the QnA Maker Cognitive Service, the QnA Maker web app, and the Azure Search Service. The QnA Maker Cognitive Service resource is only deployable in West US, but the search service and QnA Web Host app service can be deployed in other regions. So, the endpoint is always the westus endpoint except for Azure Government deployments.

Here is an example of what is created when I provision a new QnA Maker resource in the Azure Portal:
image

@GitDruid
Copy link
Author

GitDruid commented Jul 15, 2020

Hi @GitDruid, when QnA Maker is deployed, it creates three resources in Azure, the QnA Maker Cognitive Service, the QnA

Thank you @lauren-mills.
I've just started working with Botframework and it was not clear to me that "The QnA Maker Cognitive Service resource is only deployable in West US" (also looking here, it seems that there are more region then just West US: https://azure.microsoft.com/en-us/global-infrastructure/services/?products=cognitive-services).
It could be the case to add a small note/reference about it in the script doc?

@lauren-mills lauren-mills linked a pull request Jul 15, 2020 that will close this issue
3 tasks
@lauren-mills lauren-mills added Bot Services Required for internal Azure reporting. Do not delete. Do not change color. and removed Status: Backlog This will go in the product backlog labels Jul 16, 2020
@lauren-mills
Copy link
Contributor

I've added the note about QnA regions in the next branch for our next release. Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bot Services Required for internal Azure reporting. Do not delete. Do not change color. customer-replied-to Indicates that the team has replied to the issue reported by the customer. Do not delete. customer-reported Issue is created by anyone that is not a collaborator in the repository. Type: Bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants