Skip to content

Commit ed9bb08

Browse files
committed
ci: retire openai_cross_version_test + langchain_cross_version_test
Both jobs ran current CI against the frozen `integrations/openai` / `integrations/langchain` source from historical tags (databricks-ai- v0.2.0 through v0.5.0). They worked while PyPI dep evolution was backward-compatible. That contract broke with the databricks-vectorsearch → databricks-ai- search rename. The frozen historical sources import VectorSearchIndex from `databricks.vector_search.client` (the path that was removed in the rename), and there's no PR to HEAD that can update that frozen source. The matrix is permanently red regardless of what HEAD does. Per Ann: "we can restart them from the version we are deploying." Remove both jobs now; once we ship a release containing the ai-search migration (#437), the cross-version matrix can be re-introduced with that release as the new stable baseline. The bridge's own unit tests + integration tests still cover regression risk. This removes a non-Required CI signal that's now noise rather than information.
1 parent d3a4f93 commit ed9bb08

1 file changed

Lines changed: 10 additions & 88 deletions

File tree

.github/workflows/main.yml

Lines changed: 10 additions & 88 deletions
Original file line numberDiff line numberDiff line change
@@ -106,52 +106,16 @@ jobs:
106106
--exact --group tests --extra memory \
107107
pytest --cov --cov-report=term tests/unit_tests
108108
109-
langchain_cross_version_test:
110-
runs-on: ubuntu-latest
111-
name: langchain_test (${{ matrix.python-version }}, ${{ matrix.version.name }})
112-
strategy:
113-
fail-fast: false
114-
matrix:
115-
python-version: ["3.10"]
116-
version:
117-
- { ref: "databricks-ai-v0.5.0", name: "v0.5.0" }
118-
- { ref: "databricks-ai-v0.4.0", name: "v0.4.0" }
119-
- { ref: "databricks-ai-v0.3.0", name: "v0.3.0" }
120-
- { ref: "databricks-ai-v0.2.0", name: "v0.2.0" }
121-
timeout-minutes: 20
122-
steps:
123-
- name: Checkout code
124-
uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5
125-
- name: Set up Python
126-
uses: actions/setup-python@a26af69be951a213d495a4c3e4e4022e16d87065
127-
with:
128-
python-version: ${{ matrix.python-version }}
129-
- name: Install dependencies
130-
run: |
131-
pip install .
132-
- name: Checkout langchain version
133-
uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5
134-
with:
135-
ref: ${{ matrix.version.ref }}
136-
fetch-depth: 1
137-
path: older-version
138-
- name: Replace langchain with older version
139-
run: |
140-
# Remove current langchain if it exists to avoid conflicts
141-
rm -rf integrations/langchain
142-
143-
# Copy older version of langchain to the main repo
144-
cp -r older-version/integrations/langchain integrations/
145-
- name: Install langchain dependency
146-
run: |
147-
pip install integrations/langchain --group dev
148-
- name: Run tests
149-
run: |
150-
# Only testing initialization since functionality can change
151-
pytest integrations/langchain/tests/unit_tests/test_vector_search_retriever_tool.py::test_init
152-
pytest integrations/langchain/tests/unit_tests/test_genie.py
153-
pytest integrations/langchain/tests/unit_tests/test_embeddings.py
154-
pytest integrations/langchain/tests/unit_tests/test_chat_models.py
109+
# `langchain_cross_version_test` and `openai_cross_version_test` (previously
110+
# defined here) ran HEAD's CI against the frozen `integrations/langchain` /
111+
# `integrations/openai` source from historical tags (databricks-ai-v0.2.0
112+
# through v0.5.0) against current PyPI. After `databricks-vectorsearch>=0.74`
113+
# was renamed to `databricks-ai-search` and removed the `VectorSearchIndex`
114+
# re-export from `databricks.vector_search.client`, those frozen historical
115+
# sources cannot import and the matrix is permanently red regardless of
116+
# what HEAD does. Both jobs are removed; cross-version testing will be
117+
# re-seeded once a new release containing the ai-search migration ships
118+
# and we have a fresh stable baseline to test backward compat against.
155119

156120
openai_test:
157121
runs-on: ubuntu-latest
@@ -198,48 +162,6 @@ jobs:
198162
--exact --group tests --extra memory \
199163
pytest --cov --cov-report=term tests/unit_tests
200164
201-
openai_cross_version_test:
202-
runs-on: ubuntu-latest
203-
name: openai_test (${{ matrix.python-version }}, ${{ matrix.version.name }})
204-
strategy:
205-
fail-fast: false
206-
matrix:
207-
python-version: ["3.10"]
208-
version:
209-
- { ref: "databricks-ai-v0.5.0", name: "v0.5.0", mlflow: "mlflow<3" }
210-
- { ref: "databricks-ai-v0.4.0", name: "v0.4.0", mlflow: "mlflow<3" }
211-
- { ref: "databricks-ai-v0.3.0", name: "v0.3.0", mlflow: "mlflow<3" }
212-
timeout-minutes: 20
213-
steps:
214-
- name: Checkout code
215-
uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5
216-
- name: Set up Python
217-
uses: actions/setup-python@a26af69be951a213d495a4c3e4e4022e16d87065
218-
with:
219-
python-version: ${{ matrix.python-version }}
220-
- name: Checkout openai version
221-
uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5
222-
with:
223-
ref: ${{ matrix.version.ref }}
224-
fetch-depth: 1
225-
path: older-version
226-
- name: Replace openai with older version
227-
run: |
228-
# Remove current openai if it exists to avoid conflicts
229-
rm -rf integrations/openai
230-
231-
# Copy older version of openai to the main repo
232-
cp -r older-version/integrations/openai integrations/
233-
- name: Install openai dependency
234-
run: |
235-
pip install .
236-
pip install integrations/openai --group dev
237-
pip install "${{ matrix.version.mlflow }}"
238-
- name: Run tests
239-
run: |
240-
# Only testing initialization since functionality can change
241-
pytest integrations/openai/tests/unit_tests/test_vector_search_retriever_tool.py::test_vector_search_retriever_tool_init
242-
243165
llamaindex_test:
244166
runs-on: ubuntu-latest
245167
name: "llamaindex_test (python: ${{ matrix.python-version }}, uv: ${{ matrix.uv-resolution }})"

0 commit comments

Comments
 (0)