Skip to content

feat(ci): add Ollama integration test job#309

Open
crrow wants to merge 5 commits intomainfrom
issue-308-ci-ollama-integration
Open

feat(ci): add Ollama integration test job#309
crrow wants to merge 5 commits intomainfrom
issue-308-ci-ollama-integration

Conversation

@crrow
Copy link
Copy Markdown
Collaborator

@crrow crrow commented Mar 13, 2026

Summary

  • 新增 config.ci.yaml,LLM 指向 self-hosted Ollama(10.0.0.167:11434qwen2.5:32b
  • rust.yml 新增 integration job,仅 main 分支触发,跑 #[ignore] 的集成测试
  • Soak 120s / turn timeout 300s / job 限时 15min

Test plan

  • 合并后观察 main 分支 CI 中 Integration (Ollama) job 是否成功启动
  • 确认 Ollama connectivity check 通过
  • 确认 real_model_soak_test_survives_long_tape_pressure 在 CI 中通过

Closes #308

crrow and others added 5 commits March 16, 2026 21:42
Run #[ignore] integration tests against the self-hosted Ollama instance
on main branch pushes. Uses config.ci.yaml with qwen2.5:32b and
shortened soak duration (120s) for CI.

Closes #308

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Move hardcoded Ollama base_url and model to GitHub secrets
(OLLAMA_BASE_URL, OLLAMA_MODEL) so they are determined by the
runner environment, not the repository.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Use env: block instead of inline export for secret injection
- Fix connectivity check endpoint from /models to /api/tags
- Quote secret references via env vars to prevent injection
- Rename hardcoded user from 'ryan' to 'ci-bot'

Closes #381

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
…#308)

- Fix base_url missing /v1 suffix (Ollama OpenAI-compatible endpoint requires /v1)
- Add knowledge section with embedding model configuration
- Add mita heartbeat_interval required field
- Fix user name from ci-bot to ryan (integration tests hardcode this user)
- Add OLLAMA_EMBEDDING_MODEL secret to workflow secrets and envsubst step

Closes #308
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@crrow crrow force-pushed the issue-308-ci-ollama-integration branch from 556f374 to 145ceab Compare March 16, 2026 13:44
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.

feat(ci): add Ollama integration test job for release builds

1 participant