Skip to content

[CI runner] test(journey): tolerate server default RQ quantizer - #1

Open
dudanogueira wants to merge 1 commit into
mainfrom
fix/journey-default-quantizer
Open

dudanogueira wants to merge 1 commit into
mainfrom
fix/journey-default-quantizer

Conversation

@dudanogueira

Copy link
Copy Markdown
Owner

Not for merge — this PR exists so CI runs inside the fork, where re-running failed jobs is possible.

Mirrors weaviate#469. Close whenever it stops being useful.

tests-with-auth and the Docker Hub login steps are gated on !head.repo.fork and will skip here; tests-without-auth (the job that has been flaking) runs in full.

The WCD cluster this journey test targets now applies a default RQ
quantizer to HNSW indexes, so the strict deep-equal on the collection
config fails with `quantizer: undefined` vs `{type: 'rq', bits: 8,
rescoreLimit: 20}`.

Because the test connects to a fixed cloud cluster rather than the local
container, this broke every `tests-without-auth` matrix job at once, on
every Weaviate version, across all open PRs.

Pull `quantizer` out of the deep-equal and assert its shape loosely,
matching the approach already taken in fa9c488 for the openai hfresh
expectation. The rest of the config is still compared strictly.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01SXVS2uKtuEH91QEVka8zWV
@dudanogueira
dudanogueira force-pushed the fix/journey-default-quantizer branch from 6f9a442 to ac72d67 Compare September 2, 2026 21:36
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