-
Notifications
You must be signed in to change notification settings - Fork 704
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
Deployment template not working with bing-search failure #498
Comments
Same issue as I'm experiencing. "Warning BCP081: Resource type "Microsoft.Bing/accounts@2020-06-10" does not have types available." is also what I get |
Hello @yanxiaodi and @threegitty350, Sorry to hear that you are both experiencing the same deployment issue. To assist you better, could you let us know your bicep version? You can learn more here: https://learn.microsoft.com/en-us/azure/azure-resource-manager/bicep/bicep-cli |
@TaoChenOSU Thanks. The version is I just installed azure CLI yesterday so I suppose it should be the latest version. |
I guess Bicep doesn't support Bing search? I saw a warning as follows: Also I cannot find |
Adding to this: Bicep CLI version 0.21.1 > upgraded to > Bicep CLI v0.22.6 Did not result in a different outcome While running the azure deploy script I get: Warning BCP081: Resource type "Microsoft.Bing/accounts@2020-06-10" does not have types available. Using: ./deploy-azure.ps1 -Subscription {REDACTED} -DeploymentName {REDACTED} -AIService AzureOpenAI -AIApiKey {REDACTED} -AIEndpoint {REDACTED} -ResourceGroup {REDACTED} -BackendClientId {REDACTED} -FrontendClientId {REDACTED} -TenantId {REDACTED} Then it concludes with:
Deploy to azure button:
} |
This is likely due to a race condition in getting the function host key @glahaye |
Or a race condition caused by a clash of App Insights resources and config of the Bing Search Plugin Host. I have a potential fix if my theory is correct. Testing it right now. |
### Motivation and Context <!-- Thank you for your contribution to the chat-copilot repo! Please help reviewers and future users, providing the following information: 1. Why is this change required? 2. What problem does it solve? 3. What scenario does it contribute to? 4. If it fixes an open issue, please link to the issue here. --> The Web Searcher plugin requires a Bing resource, which may require elevated permission. This will fail the whole deployment, blocking developers from deploying the latest CC. Issues: #498 ### Description This PR makes deploying the Web Searcher plugin optional. And by default, it's NOT deployed. <!-- Describe your changes, the overall approach, the underlying design. These notes will help understanding how your code works. Thanks! --> ### Contribution Checklist <!-- Before submitting this PR, please make sure: --> - [ ] The code builds clean without any errors or warnings - [ ] The PR follows the [Contribution Guidelines](https://github.com/microsoft/chat-copilot/blob/main/CONTRIBUTING.md) and the [pre-submission formatting script](https://github.com/microsoft/chat-copilot/blob/main/CONTRIBUTING.md#development-scripts) raises no violations - [ ] All unit tests pass, and I have added new tests where possible - [ ] I didn't break anyone 😄
The Web Searcher plugin has become optional (default to NOT deploy) when you deploy using the deploy-azure scripts. PR: #517. Closing as the PR will address this issue. Feel free to reopen if you have further questions. |
Thanks @TaoChenOSU . But I got another error:
Not sure if I need to create a new issue? Thanks. |
The failed one is
|
@yanxiaodi We are aware of this issue and hope to have a fix today. In the meantime, you can work around the problem by simply deploying again (without deleting the resources of the failed deployment). |
### Motivation and Context <!-- Thank you for your contribution to the chat-copilot repo! Please help reviewers and future users, providing the following information: 1. Why is this change required? 2. What problem does it solve? 3. What scenario does it contribute to? 4. If it fixes an open issue, please link to the issue here. --> The Web Searcher plugin requires a Bing resource, which may require elevated permission. This will fail the whole deployment, blocking developers from deploying the latest CC. Issues: microsoft#498 ### Description This PR makes deploying the Web Searcher plugin optional. And by default, it's NOT deployed. <!-- Describe your changes, the overall approach, the underlying design. These notes will help understanding how your code works. Thanks! --> ### Contribution Checklist <!-- Before submitting this PR, please make sure: --> - [ ] The code builds clean without any errors or warnings - [ ] The PR follows the [Contribution Guidelines](https://github.com/microsoft/chat-copilot/blob/main/CONTRIBUTING.md) and the [pre-submission formatting script](https://github.com/microsoft/chat-copilot/blob/main/CONTRIBUTING.md#development-scripts) raises no violations - [ ] All unit tests pass, and I have added new tests where possible - [ ] I didn't break anyone 😄
Describe the bug
I tried two ways to deploy the app but got similar errors.
Warning BCP081: Resource type "Microsoft.Bing/accounts@2020-06-10" does not have types available.
. The script run a long time then failed.function-copichat-7tz57cfyqmyry-websearcher-plugin/default
.I suspect it's related to the bing-search component. Not sure if it's the same problem as this issue: #491
Thanks.
To Reproduce
Steps to reproduce the behavior:
https://github.com/microsoft/chat-copilot/blob/main/scripts/deploy/README.md
Expected behavior
The deployment should complete without errors.
Screenshots
If applicable, add screenshots to help explain your problem.
Platform
Additional context
Add any other context about the problem here.
The text was updated successfully, but these errors were encountered: