chore(ci): wire DEEPSEEK_API_KEY through the Harness E2E workflows - #693
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (5)
📝 WalkthroughWalkthroughHarness E2E workflows now accept an optional DeepSeek API key. Caller workflows pass the repository secret to the reusable workflow, which caches the DeepSeek provider workspace and exposes the key to source-stack and deployed-stack scenarios. ChangesDeepSeek E2E integration
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
Suggested labels: Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
skill-check — worker0 verified, 54 skipped (no docs/).
Four for four. Nicely done. |
The provider-deepseek worker (#691) resolves its credential from the DEEPSEEK_API_KEY env on the router, but the E2E workflows only forwarded the anthropic/openai/zai secrets, so a deepseek subject or judge could never discover models in CI. Declare the optional secret in the reusable workflow, forward it from every caller (including the release pipeline's E2E gate from #692), add the provider to the Rust cache workspaces, and export it in both run steps.
3995e46 to
39e06f0
Compare
Summary
deepseek_api_keysecret in the reusable_harness-e2e.ymland export it asDEEPSEEK_API_KEYin both run steps (source and registry stacks)harness-e2e-main,harness-e2e-daily,harness-e2e-deployed)provider-deepseek -> targetto the E2E Rust cache workspacesContext
provider-deepseek(#691) resolves its credential fromDEEPSEEK_API_KEYon the router; without this wiring a deepseek subject or judge never discovers models in CI (wait_for_modeltimes out). Prerequisite for switchingHARNESS_E2E_SUBJECTStodeepseek-v4-flash(judge stayszai/glm-5.2).Validated locally:
direct_answerandpersistent_statePASS withdeepseek-v4-pro; full sweep withdeepseek-v4-flashin progress.Note: the DeepSeek API now serves only
deepseek-v4-flash/deepseek-v4-pro— thedeepseek-chat/deepseek-reasoneraliases were retired upstream. Discovery picks the new ids up automatically (curated-table misses land on conservative defaults).Summary by CodeRabbit