Skip to content

ci: retire cross-version test jobs (openai + langchain)#438

Open
jennsun wants to merge 1 commit into
mainfrom
jennsun/retire-cross-version-tests
Open

ci: retire cross-version test jobs (openai + langchain)#438
jennsun wants to merge 1 commit into
mainfrom
jennsun/retire-cross-version-tests

Conversation

@jennsun

@jennsun jennsun commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Summary

Removes the langchain_cross_version_test and openai_cross_version_test jobs from .github/workflows/main.yml. Both jobs are permanently red after databricks-vectorsearch>=0.74 shipped (with the rename to databricks-ai-search and the removal of the VectorSearchIndex re-export from databricks.vector_search.client).

Targeted at unblocking the next release.

Background

The two retired jobs check out historical bridge tags (databricks-ai-v0.2.0 through v0.5.0), copy that tag's integrations/openai or integrations/langchain source over HEAD, and run a small smoke test against current PyPI. The implicit contract: "PyPI deps stay backward-compatible, so old bridge sources keep importing."

That contract broke when databricks-vectorsearch>=0.74 removed the VectorSearchIndex re-export from databricks.vector_search.client. Every historical bridge tag hard-codes:

from databricks.vector_search.client import VectorSearchIndex

That import now fails on the version of vectorsearch that the matrix install pulls. The frozen tag sources can't be patched — they're sealed in the wheel. No PR to HEAD can turn the matrix green again.

We saw this exact failure on:

What this changes

  • Delete the langchain_cross_version_test job (lines 109–154 on main).
  • Delete the openai_cross_version_test job (lines 201–241 on main).
  • Add an inline comment in the workflow explaining why both were retired and the plan for re-introducing them.

YAML validity confirmed locally via yaml.safe_load. Top-level jobs after the edit: core_test, core_test_with_extras, langchain_test, langchain_test_with_extras, openai_test, openai_test_with_extras, llamaindex_test, mcp_test, dspy_test. None of the removed jobs were marked Required.

Plan going forward

Per Ann: "we can restart them from the version we are deploying." Once a release containing the ai-search migration (#437 merged as d3a4f93) ships, we can re-introduce a cross-version matrix using that release as the new stable baseline.

The bridge's own unit tests + integration tests still cover regression risk for HEAD. This removes a non-Required CI signal that has become noise rather than information.

Test plan

  • CI on this branch turns green (no cross-version jobs to fail).
  • Verify no other workflow references cross_version_test (grep clean).
  • After merge: subsequent PRs no longer surface the three openai_test (v0.x) / four langchain_test (v0.x) failures.

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.
@jennsun jennsun force-pushed the jennsun/retire-cross-version-tests branch from ed9bb08 to 4938633 Compare June 10, 2026 00:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant