Skip to content

Update vector search index test fixtures based on cloud#5366

Merged
janniklasrose merged 8 commits into
mainfrom
janniklasrose/fix-vector-search-index-nightly
Jun 2, 2026
Merged

Update vector search index test fixtures based on cloud#5366
janniklasrose merged 8 commits into
mainfrom
janniklasrose/fix-vector-search-index-nightly

Conversation

@janniklasrose
Copy link
Copy Markdown
Contributor

@janniklasrose janniklasrose commented May 28, 2026

Changes

  • Add missing/Fix parameters to DIRECT_ACCESS indexes
    • Enforce index name (3-part UC identifier) in test server
  • Change DELTA_SYNC to DIRECT_ACCESS
    • DELTA_SYNC requires a table, which is significant overhead for tests
    • Also change recreate test to recreate based on different parameter

Why

Cloud tests are failing

Tests

@eng-dev-ecosystem-bot
Copy link
Copy Markdown
Collaborator

eng-dev-ecosystem-bot commented May 28, 2026

Commit: e9163fc

Run: 26822431424

@janniklasrose
Copy link
Copy Markdown
Contributor Author

Reran test from #5366 (comment) - all but dbr ones are passing

@janniklasrose janniklasrose changed the title Update test fixtures based on cloud Update vector search index test fixtures based on cloud Jun 1, 2026
@janniklasrose
Copy link
Copy Markdown
Contributor Author

janniklasrose commented Jun 2, 2026

Temporarily marked tests as known failures in c5ca1b7 - need to revert this post this PR

Side note: a vector search endpoint test is also marked as known failure

@janniklasrose
Copy link
Copy Markdown
Contributor Author

Some vector search tests are marked as Cloud = false - look at why

@janniklasrose janniklasrose added this pull request to the merge queue Jun 2, 2026
Merged via the queue into main with commit 865787a Jun 2, 2026
23 checks passed
@janniklasrose janniklasrose deleted the janniklasrose/fix-vector-search-index-nightly branch June 2, 2026 14:14
@eng-dev-ecosystem-bot
Copy link
Copy Markdown
Collaborator

Commit: 865787a

Run: 26825657368

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.

4 participants