Update text2vec-azure-openai to utilize isAzure: true flag and ma…

…rk `resourceName` + `deploymentId` as optional This relates to the changes in weaviate/weaviate#5776
weaviate · Sep 18, 2024 · 1952e48 · 1952e48
1 parent e333dab
commit 1952e48
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/src/collections/config/types/vectorizer.ts b/src/collections/config/types/vectorizer.ts
@@ -180,10 +180,12 @@ export type Text2VecAWSConfig = {
 export type Text2VecAzureOpenAIConfig = {
   /** The base URL to use where API requests should go. */
   baseURL?: string;
-  /** The deployment ID to use */
-  deploymentId: string;
-  /** The resource name to use. */
-  resourceName: string;
+  /** The deployment ID to use. If left empty, must be provided via X-Azure-Deployment-Id header  */
+  deploymentId?: string;
+  /** The resource name to use. If left empty, must be provided via X-Azure-Resource-Name header */
+  resourceName?: string;
+  /** Will automatically be set to true. You don't need to set this manually. */
+  isAzure?: true;
   /** Whether to vectorize the collection name. */
   vectorizeCollectionName?: boolean;
 };

diff --git a/src/collections/configure/vectorizer.ts b/src/collections/configure/vectorizer.ts
@@ -253,7 +253,10 @@ export const vectorizer = {
       vectorIndexConfig,
       vectorizerConfig: {
         name: 'text2vec-azure-openai',
-        config,
+        config: {
+          ...config,
+          isAzure: true,
+        }
       },
     });
   },