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 for cognitive services API version #508

Merged
merged 10 commits into from
Oct 18, 2023
Merged

Conversation

Dag-Calafell-MCA
Copy link
Contributor

Motivation and Context

Fix for issue #425

The older API version was no longer working. It allows for deploying a new Azure OpenAI resource.

Description

Updates the API version and SKU of the Azure OpenAI

Contribution Checklist

@github-actions github-actions bot added the deployment Issues related to deploying Chat-Copilot label Oct 16, 2023
@Dag-Calafell-MCA
Copy link
Contributor Author

@microsoft-github-policy-service agree

crickman
crickman previously approved these changes Oct 17, 2023
glahaye
glahaye previously approved these changes Oct 17, 2023
@glahaye glahaye added this pull request to the merge queue Oct 18, 2023
@glahaye glahaye removed this pull request from the merge queue due to a manual request Oct 18, 2023
Updated with latest from main
Changed Microsoft.CognitiveServices/accounts to be 2023-05-01 too
Removed scaleSettings
Changed capacity from 20 to 30
@glahaye glahaye dismissed stale reviews from crickman and themself via f18736c October 18, 2023 23:09
scripts/deploy/main.bicep Outdated Show resolved Hide resolved
scripts/deploy/main.bicep Outdated Show resolved Hide resolved
scripts/deploy/main.bicep Outdated Show resolved Hide resolved
@glahaye glahaye added this pull request to the merge queue Oct 18, 2023
Merged via the queue into microsoft:main with commit b0801da Oct 18, 2023
Ryangr0 pushed a commit to webgrip/chat-copilot that referenced this pull request Oct 26, 2023
### Motivation and Context

Fix for issue microsoft#425 

The older API version was no longer working. It allows for deploying a
new Azure OpenAI resource.

### Description

Updates the API version and SKU of the Azure OpenAI 

### Contribution Checklist

- [x] The code builds clean without any errors or warnings
- [x] 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
- [x] All unit tests pass, and I have added new tests where possible
- [x] I didn't break anyone 😄

---------

Co-authored-by: Gil LaHaye <gillahaye@microsoft.com>
teamleader-dev pushed a commit to vlink-group/chat-copilot that referenced this pull request Oct 7, 2024
### Motivation and Context

Fix for issue microsoft#425 

The older API version was no longer working. It allows for deploying a
new Azure OpenAI resource.

### Description

Updates the API version and SKU of the Azure OpenAI 

### Contribution Checklist

- [x] The code builds clean without any errors or warnings
- [x] 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
- [x] All unit tests pass, and I have added new tests where possible
- [x] I didn't break anyone 😄

---------

Co-authored-by: Gil LaHaye <gillahaye@microsoft.com>
kb0039 pushed a commit to aaronba/chat-copilot that referenced this pull request Jan 8, 2025
### Motivation and Context

Fix for issue microsoft#425 

The older API version was no longer working. It allows for deploying a
new Azure OpenAI resource.

### Description

Updates the API version and SKU of the Azure OpenAI 

### Contribution Checklist

- [x] The code builds clean without any errors or warnings
- [x] 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
- [x] All unit tests pass, and I have added new tests where possible
- [x] I didn't break anyone 😄

---------

Co-authored-by: Gil LaHaye <gillahaye@microsoft.com>
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.

4 participants