Skip to content

Conversation

@ckittel
Copy link
Member

@ckittel ckittel commented Apr 23, 2025

  • Update model from gpt35-turbo (Standard) to gpt-4o-mini (Standard)
  • Replace direct Azure OpenAI deployment with Azure AI Services deployment
  • A few minor Resource Manager API updates
  • Update to a more strict content policy filter
  • Remove key access to Azure Storage
  • Add cors configuration for Azure Storage
@ckittel ckittel requested a review from Copilot April 23, 2025 21:31
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the deployment configuration to use the new gpt-4o-mini model and Azure AI Services instead of the previous gpt35-turbo and direct Azure OpenAI deployment. Key changes include updated connection and deployment names in the documentation, adjustments to the Resource Manager API usage, and updates to the Azure Storage configuration.

Files not reviewed (4)
  • infra-as-code/bicep/machinelearning.bicep: Language not supported
  • infra-as-code/bicep/main.bicep: Language not supported
  • infra-as-code/bicep/openai.bicep: Language not supported
  • infra-as-code/bicep/storage.bicep: Language not supported
Comments suppressed due to low confidence (1)

README.md:133

  • The description still refers to 'Azure OpenAI instance' even after changing the connection name to 'azureaiservices_aoai'. Consider updating the description to 'Azure AI Services instance' for clarity and consistency.
- For **Connection**, select 'azureaiservices_aoai' from the dropdown menu. This is your deployed Azure OpenAI instance. 
Copy link
Contributor

@ferantivero ferantivero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great edits @ckittel LGTM 🚢

@ckittel ckittel merged commit 9be6076 into main Apr 24, 2025
3 checks passed
@ckittel ckittel deleted the ckittel/ai-services branch April 24, 2025 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants