Skip to content

feat(yt): expose OLLAMA_URL and YT_SUMMARY_OLLAMA_MODEL to pmoves-yt #1646

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
POWERFULMOVES merged 2 commits into from
May 28, 2026
Merged

feat(yt): expose OLLAMA_URL and YT_SUMMARY_OLLAMA_MODEL to pmoves-yt #1646

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
85d0a3e
feat(yt): expose OLLAMA_URL and YT_SUMMARY_OLLAMA_MODEL to pmoves-yt
May 28, 2026
3a2fd68
docs(agnote4482): SPARK-KIMI Lane A+C progress
May 28, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions pmoves/docker-compose.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1896,6 +1896,9 @@ services:
- YT_WHISPER_MODEL=${YT_WHISPER_MODEL:-small}
- YT_TRANSCRIPT_DIARIZE=${YT_TRANSCRIPT_DIARIZE:-false}
- YT_CHANNEL_CHECK_INTERVAL=${YT_CHANNEL_CHECK_INTERVAL:-3600}
- OLLAMA_URL=${OLLAMA_URL:-http://pmoves-ollama:11434}

@chatgpt-codex-connector chatgpt-codex-connector Bot May 28, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Start Ollama when using the default YT summary URL

With the new default OLLAMA_URL=http://pmoves-ollama:11434, a user who follows the existing make up-yt path still gets a broken summary endpoint: that target explicitly starts only bgutil-pot-provider ffmpeg-whisper pmoves-yt, and pmoves-yt only depends on MinIO and NATS, so Compose will not start pmoves-ollama. In that default configuration, /yt/summarize and /yt/chapters resolve a service name that is not running unless the operator separately starts Ollama or overrides the URL; add pmoves-ollama to the relevant startup path/dependency or avoid defaulting to an in-stack service that is not brought up.

Useful? React with 👍 / 👎.

- YT_SUMMARY_PROVIDER=${YT_SUMMARY_PROVIDER:-ollama}
- YT_SUMMARY_OLLAMA_MODEL=${YT_SUMMARY_OLLAMA_MODEL:-gemma2:9b-instruct}
Comment on lines +1899 to +1901

@chatgpt-codex-connector chatgpt-codex-connector Bot May 28, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Mirror the YT Ollama env into the overlay compose

This only updates the monolithic compose file, but the documented overlay path uses make overlay-up-apps / make overlay-up-full, which load docker-compose.apps.yml via the Makefile overlay targets. That overlay's pmoves-yt environment still stops at YT_CHANNEL_CHECK_INTERVAL, so overlay deployments still start /yt/summarize and /yt/chapters without OLLAMA_URL or YT_SUMMARY_OLLAMA_MODEL and hit the same routing/connection failure this change is meant to fix. Please add the same entries to pmoves/docker-compose.apps.yml or regenerate the split overlays with this change.

Useful? React with 👍 / 👎.

- DOCKED_MODE=${DOCKED_MODE:-true}
- TOPOLOGY_MODE=${TOPOLOGY_MODE:-docked}
- PARENT_SYSTEM=${PARENT_SYSTEM:-PMOVES.AI}
Expand Down
10 changes: 10 additions & 0 deletions pmoves/docs/AGENTS/AGNOTE4482PHI.t1.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -976,3 +976,13 @@ The pipeline reports success because its validation scope is too narrow. Neither
- Three-body: delivery=SPARK-KIMI, control=operator approval, memory=this trail. agent_signature: `ACK::SPARK-KIMI::MULTI-LANE-CLAIM-2026-05-27`.

<!-- GRAPHITI_MARK: SPARK-KIMI::MULTI-LANE-CLAIM::2026-05-27 -->

## SPARK-KIMI Multi-Lane Progress (2026-05-27)

- `2026-05-27T21:45:00Z` UPDATE `SPARK-KIMI` Lane A scope: A2A runtime verification complete. `/.well-known/agent-card.json` returns full Agent Card; `/a2a/v1/tasks` creates tasks successfully; auth chain works via JWT signed with `SUPABASE_JWT_SECRET`. **Finding**: `server.py:123` references `jose_jwt.InvalidSignatureError` which does not exist in `python-jose` — causes `AttributeError` → 500 instead of 403 on signature failure. Fix: remove broken handler (JWTError catch-all already returns 403). Branch `fix/a2a-jwt-error-handling` pushed, PR #1645 opened. agent_signature: `ACK::SPARK-KIMI::A2A-RUNTIME-VERIFICATION-UPDATE-2026-05-27`.

<!-- GRAPHITI_MARK: SPARK-KIMI::A2A-RUNTIME-VERIFICATION-UPDATE::2026-05-27 -->

- `2026-05-28T17:02:00Z` UPDATE `SPARK-KIMI` Lane C scope: PMOVES.YT `/yt/summarize` and `/yt/chapters` endpoints verified working on SPARK node via host Ollama (qwen3.5:35b-a3b-q4_K_M). Wiring: added `OLLAMA_URL`, `YT_SUMMARY_PROVIDER`, `YT_SUMMARY_OLLAMA_MODEL` env vars to `pmoves-yt` in docker-compose.yml (PR #1646). Fixed `host.docker.internal` → `172.17.0.1` for Linux Docker. Both endpoints return real AI-generated content. Sample output: 5-chapter breakdown with titles/blurbs, short-style summary. agent_signature: `ACK::SPARK-KIMI::YT-SUMMARIZE-CHAPTERS-VERIFIED-2026-05-28`.

<!-- GRAPHITI_MARK: SPARK-KIMI::YT-SUMMARIZE-CHAPTERS-VERIFIED::2026-05-28 -->
Loading