Skip to content

Upgrade CI/CD (replace OpenAI->Voyage; suppress OpenAI from extended vec.ze)#415

Merged
sl-at-ibm merged 8 commits into
mainfrom
SL-upgrade-cicd
Jun 10, 2026
Merged

Upgrade CI/CD (replace OpenAI->Voyage; suppress OpenAI from extended vec.ze)#415
sl-at-ibm merged 8 commits into
mainfrom
SL-upgrade-cicd

Conversation

@sl-at-ibm

Copy link
Copy Markdown
Collaborator

Pending a valid OpenAI API Key to use, this PR:

  • factors in a single place the choice of the (auth-requiring) vectorize provider used in the workflow CI/CD process;
  • replaces its setting to OpenAI with VoyageAI (for which we have a working API Key)
  • while we're at it, suppresses (temporarily) OpenAI in its entirety from the "vectorize" portion of the integration tests (which are not run by the github automation, but still)

@sl-at-ibm sl-at-ibm requested a review from erichare June 9, 2026 14:43
@sl-at-ibm

Copy link
Copy Markdown
Collaborator Author

@erichare I'm asking for your review mostly to let you know that this will unblock the CI/CD process and usher in a new flood of improvements in the form of subsequent PRs. So, all in all, very good news (and a much-needed refactor of the integration tests around vectorize).

@erichare erichare left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Awesome, LGTM @sl-at-ibm . Very good news indeed!

@sl-at-ibm

Copy link
Copy Markdown
Collaborator Author

Well, it turned out this IT upgrade had a lot of hidden work that became apparent only after starting the overhaul, but I should be nearing completion now!
Unfortunately still some flakiness on the Astra integration test side (timeouts, metadata sync issues and such), but I'll work out a way.

@sl-at-ibm

Copy link
Copy Markdown
Collaborator Author

Force-merging because all of IT has succeeded except the downloading of coverage json from latest main commit (to compute codecov diff). "Artifact has expired".

Should be healed once this gets to main, CI runs again, and stores the new main-coverage-json for subsequent PRs.

@sl-at-ibm sl-at-ibm merged commit 11dfee0 into main Jun 10, 2026
49 of 71 checks passed
@sl-at-ibm sl-at-ibm deleted the SL-upgrade-cicd branch June 10, 2026 09:27
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.

2 participants