Skip to content

chore: Replace text-embedding-ada-002 with text-embedding-3-small#141

Merged
Prajwal-Microsoft merged 4 commits intodevfrom
psl-embedding-modelchage
Mar 27, 2026
Merged

chore: Replace text-embedding-ada-002 with text-embedding-3-small#141
Prajwal-Microsoft merged 4 commits intodevfrom
psl-embedding-modelchage

Conversation

@NirajC-Microsoft
Copy link
Copy Markdown
Contributor

Purpose

Replace text-embedding-ada-002 with text-embedding-3-small

Summary

Migrates BYOCC from text-embedding-ada-002 to text-embedding-3-small to align with supported models and improve long-term maintainability.

Both models default to 1536 dimensions — this is a drop-in replacement with no index schema changes.


Files Changed

# File Change
1 main.bicep @allowed + default → text-embedding-3-small, version '2' → '1'
2 main_custom.bicep Same as above
3 main.bicep @allowed + default → text-embedding-3-small, updated quota comment
4 main.json defaultValue + allowedValues → text-embedding-3-small, version "2" → "1"
5 main.json defaultValue + allowedValues → text-embedding-3-small
6 checkquota.sh Quota key → OpenAI.GlobalStandard.text-embedding-3-small
7 deploy.yml Quota failure email model name updated
8 01_create_products_search_index.py Parameterized model_name in get_embeddings_batch(), removed hardcoded model ID
9 02_create_policies_search_index.py Parameterized model_name in get_embeddings(), removed hardcoded model ID
10 CustomizingAzdParameters.md Default value updated
11 AzureGPTQuotaSettings.md Model name reference updated

Testing Notes

  • Search for "text-embedding-ada-002" after changes and does not found it.
  • Deployed in multiple ways and performed GP testing successfully:
    • WAF
    • NON-WAF
    • Custom Deployment (ARM)
    • Local deployment (main_custom)
  • Verified model and version in AI foundry after each deployent.
  • Quota check script → Ran successfully

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

@NirajC-Microsoft NirajC-Microsoft changed the title Psl embedding modelchage refactor: Replace text-embedding-ada-002 with text-embedding-3-small Mar 18, 2026
@NirajC-Microsoft NirajC-Microsoft changed the title refactor: Replace text-embedding-ada-002 with text-embedding-3-small chore: Replace text-embedding-ada-002 with text-embedding-3-small Mar 18, 2026

This comment was marked as resolved.

@Prajwal-Microsoft Prajwal-Microsoft merged commit 84c7c71 into dev Mar 27, 2026
13 of 14 checks passed
@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 1.2.5 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants