fix: entrypoint-llama.sh install_project_crons ignores DISINTO_AGENTS — installs all agents #310

New issue

Closed

opened 2026-04-06 18:18:44 +00:00 by dev-bot · 0 comments

dev-bot commented

2026-04-06 18:18:44 +00:00

Collaborator

Problem

entrypoint-llama.sh calls install_project_crons() which installs cron entries for review-poll, dev-poll, and gardener. This caused both containers to run review-poll simultaneously, producing duplicate reviews and error comments on PRs #305 and #307.

Root cause

entrypoint-llama.sh was updated to call install_project_crons (likely from #231 or a related PR). The llama container should only run dev-poll via its while loop — no cron at all.

Cron is not a suitable runtime for the llama container because it does not inherit Docker compose environment variables (ANTHROPIC_API_KEY, ANTHROPIC_BASE_URL, CLAUDE_CONFIG_DIR are lost). The while loop in entrypoint-llama.sh inherits these from PID 1.

Additionally, the local model is only capable of dev work — review and gardener require a more capable model and run in the Anthropic agents container.

Fix

Remove the install_project_crons call and the cron daemon startup from entrypoint-llama.sh. The while loop is the only execution path.

Affected files

docker/agents/entrypoint-llama.sh (remove install_project_crons call and cron startup)

Acceptance criteria

entrypoint-llama.sh does not install any cron entries
entrypoint-llama.sh does not start the cron daemon
dev-poll runs via the while loop only
No duplicate reviews when both containers are running

## Problem entrypoint-llama.sh calls install_project_crons() which installs cron entries for review-poll, dev-poll, and gardener. This caused both containers to run review-poll simultaneously, producing duplicate reviews and error comments on PRs #305 and #307. ## Root cause entrypoint-llama.sh was updated to call install_project_crons (likely from #231 or a related PR). The llama container should only run dev-poll via its while loop — no cron at all. Cron is not a suitable runtime for the llama container because it does not inherit Docker compose environment variables (ANTHROPIC_API_KEY, ANTHROPIC_BASE_URL, CLAUDE_CONFIG_DIR are lost). The while loop in entrypoint-llama.sh inherits these from PID 1. Additionally, the local model is only capable of dev work — review and gardener require a more capable model and run in the Anthropic agents container. ## Fix Remove the install_project_crons call and the cron daemon startup from entrypoint-llama.sh. The while loop is the only execution path. ## Affected files - docker/agents/entrypoint-llama.sh (remove install_project_crons call and cron startup) ## Acceptance criteria - [ ] entrypoint-llama.sh does not install any cron entries - [ ] entrypoint-llama.sh does not start the cron daemon - [ ] dev-poll runs via the while loop only - [ ] No duplicate reviews when both containers are running