Skip to content

chore: upgrade Azure OpenAI embedding model to text-embedding-3-small#2128

Merged
Roopan-Microsoft merged 6 commits intoAzure-Samples:devfrom
Ayaz-Microsoft:textEmbedding-small
Apr 6, 2026
Merged

chore: upgrade Azure OpenAI embedding model to text-embedding-3-small#2128
Roopan-Microsoft merged 6 commits intoAzure-Samples:devfrom
Ayaz-Microsoft:textEmbedding-small

Conversation

@Ayaz-Microsoft
Copy link
Copy Markdown
Contributor

Purpose

  • ...Upgraded text embedding model to text-embedding-3-small

Does this introduce a breaking change?

  • Yes
  • No

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install
  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

Copy link
Copy Markdown
Contributor

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

Updates the solution’s default Azure OpenAI embedding model configuration to use text-embedding-3-small, aligning infrastructure parameters and local environment defaults with the newer embedding deployment.

Changes:

  • Switched default embedding model/name from text-embedding-ada-002 to text-embedding-3-small and set model version to 1 in infra parameters/templates.
  • Updated backend env default and .env.sample to use text-embedding-3-small.
  • Minor infra adjustments: safer module output access for MI principal ID, and suppressed a Bicep linter warning for function keys.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
infra/modules/core/ai/cognitiveservices.bicep Adjusts MI principalId output handling (now uses optional access + fallback).
infra/modules/app/function.bicep Adds #disable-next-line BCP081 for the functionKeys resource.
infra/main.parameters.json Updates embedding model/name/version defaults to text-embedding-3-small / 1.
infra/main.json Regenerated ARM template reflecting updated defaults and safer output access.
infra/main.bicep Updates embedding model/name/version default params; minor formatting change for tags union.
code/backend/batch/utilities/helpers/env_helper.py Changes default AZURE_OPENAI_EMBEDDING_MODEL fallback to text-embedding-3-small.
.env.sample Updates sample embedding model env var to text-embedding-3-small.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread infra/main.parameters.json
Comment thread infra/main.bicep
Comment thread infra/modules/core/ai/cognitiveservices.bicep Outdated
Comment thread infra/modules/app/function.bicep Outdated
Copy link
Copy Markdown
Contributor

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

Copilot reviewed 24 out of 24 changed files in this pull request and generated 4 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/broken-links-checker.yml
Comment thread docs/azure_openai_model_quota_settings.md
Comment thread docs/TEAMS_LOCAL_DEPLOYMENT.md
Comment thread docs/QuotaCheck.md
Copy link
Copy Markdown
Contributor

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

Copilot reviewed 24 out of 24 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Roopan-Microsoft Roopan-Microsoft merged commit d5f93d0 into Azure-Samples:dev Apr 6, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants