Skip to content
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.

Sign up for GitHub

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

Jump to bottom

OpenAI Deployment is Broken #516

Closed
victor-v-rad opened this issue Oct 17, 2023 · 3 comments
Closed

OpenAI Deployment is Broken #516

victor-v-rad opened this issue Oct 17, 2023 · 3 comments
Assignees
@glahaye
Labels
bug Something isn't working

Comments

@victor-v-rad
Copy link

victor-v-rad commented Oct 17, 2023 •
edited
Loading

To Reproduce
Steps to reproduce the behavior:

  1. Go to https://github.com/microsoft/chat-copilot/tree/main/scripts/deploy
  2. Click on "Deploy to Azure" button
  3. Enter OpenAI in Ai Service field (screenshot provided)
  4. Deploy
  5. Open deployed Web App Configuration
  6. See api keys are set to AzureOpenAl but not to OpenAl (screenshot provided)
  7. Open webapi website and see the error (screenshot provided)
  8. Open Application Event Logs and see APIKey error

Expected behavior

  • api keys should be set for OpenAl
  • frontend should open without the errors

Screenshots
Screenshot 2023-10-17 at 1 19 53 PM
Screenshot 2023-10-17 at 1 20 02 PM

image
image
image

Platform

  • Language: C#
  • Source: Latest code and latest released Chat Copilot version
@victor-v-rad victor-v-rad changed the title OpenAI Deployment is broken OpenAI Deployment is Broken Oct 17, 2023
@victor-v-rad victor-v-rad mentioned this issue Oct 17, 2023
Merged
3 tasks
@glahaye glahaye self-assigned this Oct 17, 2023
@glahaye glahaye added the bug Something isn't working label Oct 17, 2023
@glahaye
Copy link
Collaborator

glahaye commented Oct 17, 2023

Thanks for bringing this issue to our attention.

Please see PR #519 addressing it.

@glahaye
Copy link
Collaborator

glahaye commented Oct 18, 2023

@victor-v-rad Closing this as fixed now.

If ever you hit any further problems, please let us know again!

Thanks.

@glahaye glahaye closed this as completed Oct 18, 2023
@victor-v-rad
Copy link
Author

Wow, fixed in several hours after I reported it! Thank you @glahaye, greatly appreciated.

gormaar pushed a commit to equinor/chat-copilot that referenced this issue Nov 29, 2023
@adrianwyatt
Add error propagation for invalid completion configuration (microsoft…
e5b3934 
…#516)

### Motivation and Context
When the Completion config is invalid, we end up repeating the user's
message back to them instead of showing an error.


### Description
- Added propagation of a failed SK invoke back to the top-level
controller call.
- Enabled AI exception detail messages
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@glahaye glahaye

Labels
bug Something isn't working
Projects
No open projects
Apps & Services Semantic Kernel
Status: Done
Milestone
No milestone
Development

No branches or pull requests
2 participants
@victor-v-rad @glahaye