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.

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

Fix deployment when using OpenAI (as opposed to AOAI) #519

Merged
merged 1 commit into from
Oct 18, 2023

Conversation

glahaye
Copy link
Collaborator

@glahaye glahaye commented Oct 17, 2023

Motivation and Context

Using the deployment scripts or Deploy to Azure button is broken when using an OpenAI account (as opposed to an AOAI one).

Description

Create the proper settings in the bicep/ARM template

Contribution Checklist

@glahaye glahaye self-assigned this Oct 17, 2023
@github-actions github-actions bot added the deployment Issues related to deploying Chat-Copilot label Oct 17, 2023
@glahaye glahaye added this pull request to the merge queue Oct 18, 2023
Merged via the queue into microsoft:main with commit 0907bd1 Oct 18, 2023
5 checks passed
@glahaye glahaye deleted the fix_openai_deploy branch October 18, 2023 00:05
Ryangr0 pushed a commit to webgrip/chat-copilot that referenced this pull request Oct 26, 2023
### Motivation and Context
Using the deployment scripts or Deploy to Azure button is broken when
using an OpenAI account (as opposed to an AOAI one).

### Description
Create the proper settings in the bicep/ARM template

### Contribution Checklist
- [ ] 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deployment Issues related to deploying Chat-Copilot
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants