Remove Azure US East Config and Credentials #288

davidgxue · 2024-01-30T19:48:40Z

We have now migrated Ask Astro credentials into a single account on Azure. All OpenAI model deployments now use the same Azure endpoint (US East 2) and credentials. Therefore, the US East credentials environment variable and the config field is no longer relevant.

New credentials already tested in dev servers. AZURE_OPENAI_USEAST_PARAMS and AZURE_OPENAI_USEAST2_PARAMS was set to the be same
In this branch, I removed the environment variable for AZURE_OPENAI_USEAST_PARAMS and the service still worked as expected.

Related to Issue #197 (Azure OpenAI migration, not OpenAI)

api/ask_astro/clients/weaviate_.py

Remove Azure US East credentials

541fbf1

davidgxue added this to the 0.3.0 milestone Jan 30, 2024

davidgxue self-assigned this Jan 30, 2024

davidgxue requested review from Lee-W, pankajastro and sunank200 as code owners January 30, 2024 19:48

Lee-W approved these changes Jan 31, 2024

View reviewed changes

sunank200 reviewed Jan 31, 2024

View reviewed changes

api/ask_astro/clients/weaviate_.py Show resolved Hide resolved

davidgxue requested a review from sunank200 January 31, 2024 05:35

davidgxue merged commit 6c81744 into main Jan 31, 2024
8 checks passed

davidgxue deleted the migrate_azure_deployment branch January 31, 2024 23:31

Provide feedback