Skip to content

ci: run integration tests against Weaviate 1.39.0-rc.0#2099

Open
jfrancoa wants to merge 1 commit into
mainfrom
bump-integration-tests-1.39
Open

ci: run integration tests against Weaviate 1.39.0-rc.0#2099
jfrancoa wants to merge 1 commit into
mainfrom
bump-integration-tests-1.39

Conversation

@jfrancoa

Copy link
Copy Markdown
Contributor

Summary

Moves the integration-test matrix from $WEAVIATE_136 (1.36.12) to a new $WEAVIATE_139 (1.39.0-rc.0) and adds the version to the test-package matrix.

This is split out of #1991 so that the version bump and its fallout can be reviewed on their own, rather than mixed into the alter-schema drop-vector-index feature. #1991 is stacked on top of this branch.

Why now

The drop-vector-index endpoint only exists from 1.39, so its tests cannot run in CI until the matrix moves. Independently, the matrix has been sitting on 1.36.x while $WEAVIATE_137 was already defined, so a range of version-gated tests never actually executed — 56 more tests run at 1.39 than at 1.36.12 (1072 passed vs 1016).

Pinned tag

There is no plain 1.39.0-rc.0 tag on Docker Hub yet, only commit-suffixed builds, so the newest one is pinned following the existing WEAVIATE_137 convention. Worth swapping for the clean tag once the RC is published.

Test change

One existing test needed adapting to an intentional server change. From 1.39 asyncEnabled is no longer stored per class but derived server-side as factor > 1 && !globallyDisabled (adapters/handlers/rest/restcompat/wrappers.go). Three assertion sites use a replication factor of 1 and therefore now expect False.

Rather than sprinkling version checks, the scattered is_at_least(1, 26, 0) branches are replaced by one _expected_async_enabled(version, factor) helper that encodes both the old and new server behaviour. The two sites using factor=2 are unaffected — factor > 1 still yields True there — and are left alone.

No coverage is removed: the pre-1.39 expectations remain asserted on older servers.

Verification

The full integration suite was run against 1.36.12 and against 1.39.0-rc.0 on an otherwise identical local stack built from ci/docker-compose.yml. Both produce identical failure sets (109 entries, all from compose instances not available locally), so the bump introduces no regressions in the ~1000 tests that environment can exercise.

Two caveats for reviewers:

  • Tests requiring the other compose instances (RBAC, cluster, proxy, okta, backup, namespaces) failed identically at both versions and were effectively not compared. CI is the first real check on those.
  • The comparison was run on ARM using the .arm64 image variants; CI uses the .amd64 tags pinned here.

🤖 Generated with Claude Code

@jfrancoa
jfrancoa requested a review from a team as a code owner July 22, 2026 12:04

@orca-security-eu orca-security-eu Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Infrastructure as Code high 0   medium 0   low 0   info 0 View in Orca
Passed Passed SAST high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Vulnerabilities high 0   medium 0   low 0   info 0 View in Orca

Bumps the integration-test matrix from 1.36.x to 1.39.0-rc.0 and adds the
version to the test-package matrix. There is no plain `1.39.0-rc.0` tag on
Docker Hub yet, only commit-suffixed builds, so the newest one is pinned
following the existing `WEAVIATE_137` convention.

One existing test needed adapting to an intentional server change in 1.39:
`asyncEnabled` is no longer stored per class but derived server-side as
`factor > 1 && !globallyDisabled`. `test_collection_config_full` uses a
replication factor of 1, so the field now reports `False`. The pre-1.39
assertions are kept behind a version check.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@jfrancoa
jfrancoa force-pushed the bump-integration-tests-1.39 branch from 736a7cd to be6c8dc Compare July 22, 2026 12:05
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