Skip to content

Commit feb4bd7

Browse files
Python: fix Azure OpenAI integration setup
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 677bc31 commit feb4bd7

4 files changed

Lines changed: 4 additions & 8 deletions

File tree

.github/workflows/python-integration-tests.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -101,6 +101,7 @@ jobs:
101101
AZURE_OPENAI_DEPLOYMENT_NAME: ${{ vars.AZUREOPENAI__RESPONSESDEPLOYMENTNAME }}
102102
AZURE_OPENAI_EMBEDDING_DEPLOYMENT_NAME: ${{ vars.AZUREOPENAI__EMBEDDINGDEPLOYMENTNAME }}
103103
AZURE_OPENAI_ENDPOINT: ${{ vars.AZUREOPENAI__ENDPOINT }}
104+
AZURE_OPENAI_API_VERSION: ${{ vars.AZUREOPENAI__APIVERSION || '2024-12-01-preview' }}
104105
defaults:
105106
run:
106107
working-directory: python

.github/workflows/python-merge-tests.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -200,6 +200,7 @@ jobs:
200200
AZURE_OPENAI_DEPLOYMENT_NAME: ${{ vars.AZUREOPENAI__RESPONSESDEPLOYMENTNAME }}
201201
AZURE_OPENAI_EMBEDDING_DEPLOYMENT_NAME: ${{ vars.AZUREOPENAI__EMBEDDINGDEPLOYMENTNAME }}
202202
AZURE_OPENAI_ENDPOINT: ${{ vars.AZUREOPENAI__ENDPOINT }}
203+
AZURE_OPENAI_API_VERSION: ${{ vars.AZUREOPENAI__APIVERSION || '2024-12-01-preview' }}
203204
defaults:
204205
run:
205206
working-directory: python

python/packages/azure-ai/tests/azure_openai/test_azure_embedding_client.py

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -108,10 +108,7 @@ def test_azure_otel_provider_name(azure_embedding_unit_test_env: None) -> None:
108108

109109
skip_if_azure_openai_integration_tests_disabled = pytest.mark.skipif(
110110
os.getenv("AZURE_OPENAI_ENDPOINT", "") in ("", "https://test-endpoint.com")
111-
or (
112-
os.getenv("AZURE_OPENAI_EMBEDDING_DEPLOYMENT_NAME", "") == ""
113-
and os.getenv("AZURE_OPENAI_DEPLOYMENT_NAME", "") == ""
114-
),
111+
or os.getenv("AZURE_OPENAI_EMBEDDING_DEPLOYMENT_NAME", "") == "",
115112
reason="No Azure OpenAI endpoint or embedding deployment provided; skipping integration tests.",
116113
)
117114

python/packages/openai/tests/openai/test_openai_embedding_client_azure.py

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,10 +17,7 @@
1717

1818
skip_if_azure_openai_integration_tests_disabled = pytest.mark.skipif(
1919
os.getenv("AZURE_OPENAI_ENDPOINT", "") in ("", "https://test-endpoint.openai.azure.com")
20-
or (
21-
os.getenv("AZURE_OPENAI_EMBEDDING_DEPLOYMENT_NAME", "") == ""
22-
and os.getenv("AZURE_OPENAI_DEPLOYMENT_NAME", "") == ""
23-
),
20+
or os.getenv("AZURE_OPENAI_EMBEDDING_DEPLOYMENT_NAME", "") == "",
2421
reason="No real Azure OpenAI endpoint or embedding deployment provided; skipping integration tests.",
2522
)
2623

0 commit comments

Comments
 (0)