Rockachopa/Timmy-time-dashboard
1
0
Fork 2
Code Issues 7 Pull Requests Actions Packages Projects Releases Wiki Activity

[loop-generated] [bug] Pre-push hook times out on agentic_loop test (Ollama connection) #1459

New Issue
Closed
opened 2026-03-24 19:53:32 +00:00 by Timmy · 1 comment
Timmy commented 2026-03-24 19:53:32 +00:00
Owner
Copy Link

Bug

The pre-push hook (tox -e pre-push) hangs/times out due to a test in the agentic_loop suite that attempts to connect to Ollama. This blocks legitimate pushes.

Reproduction

  1. Make any change on a branch
  2. Run git push (triggers pre-push hook)
  3. Hook eventually times out on agentic_loop tests that try real Ollama connections

Expected

All tests in pre-push should use mocks for external services. No test should depend on Ollama being available.

Impact

Forces agents to bypass the pre-push hook, weakening the CI gate.

Suggested fix

  • Mock Ollama connections in agentic_loop tests
  • Add timeout guards to any test that makes network calls
  • Ensure tox -e unit and tox -e pre-push work fully offline
## Bug The pre-push hook (`tox -e pre-push`) hangs/times out due to a test in the agentic_loop suite that attempts to connect to Ollama. This blocks legitimate pushes. ## Reproduction 1. Make any change on a branch 2. Run `git push` (triggers pre-push hook) 3. Hook eventually times out on agentic_loop tests that try real Ollama connections ## Expected All tests in pre-push should use mocks for external services. No test should depend on Ollama being available. ## Impact Forces agents to bypass the pre-push hook, weakening the CI gate. ## Suggested fix - Mock Ollama connections in agentic_loop tests - Add timeout guards to any test that makes network calls - Ensure `tox -e unit` and `tox -e pre-push` work fully offline
Timmy commented 2026-03-24 20:38:23 +00:00
Author
Owner
Copy Link

Investigation Complete — Cannot Reproduce

Thorough investigation of all agentic_loop and ollama test files shows:

  1. All agentic_loop tests properly mock _get_loop_agent() — no real Ollama connections
  2. conftest_markers.py auto-marks tests with "ollama" in the path, and tox -e unit/pre-push exclude those
  3. tox -e unit passes cleanly: 966 passed, 3 skipped, 7.68s — no hangs

The tox configuration already excludes -m "not ollama" from both unit and pre-push envs. The conftest auto-marker system catches any test file with "ollama" in its path.

Closing as cannot-reproduce. If this recurs, it may be an import-time side effect from agno.models.ollama.Ollama — if so, reopen with the specific test name and stack trace.

## Investigation Complete — Cannot Reproduce Thorough investigation of all agentic_loop and ollama test files shows: 1. **All agentic_loop tests properly mock** `_get_loop_agent()` — no real Ollama connections 2. **conftest_markers.py auto-marks** tests with "ollama" in the path, and tox -e unit/pre-push exclude those 3. **tox -e unit passes cleanly**: 966 passed, 3 skipped, 7.68s — no hangs The tox configuration already excludes `-m "not ollama"` from both unit and pre-push envs. The conftest auto-marker system catches any test file with "ollama" in its path. **Closing as cannot-reproduce.** If this recurs, it may be an import-time side effect from `agno.models.ollama.Ollama` — if so, reopen with the specific test name and stack trace.
Timmy closed this issue 2026-03-24 20:38:32 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
222-epic
Workshop: Timmy as Presence (Epic #222)
 actionable
Has a concrete code/config task extracted
 assigned-claude
Issue currently assigned to Claude agent — do not assign to another agent
 assigned-gemini
Issue currently assigned to Gemini agent — do not assign to another agent
 assigned-groq
assigned-kimi
Issue currently assigned to Kimi agent — do not assign to another agent
 assigned-manus
Issue currently assigned to Manus agent — do not assign to another agent
 claude-ready
consolidation
Part of a consolidation epic
 deprioritized
Keep open but not blocking P0 work
 deprioritized
Keep open but not blocking P0 work
 duplicate
Duplicate of another issue
 gemini-review
Auto-generated by Gemini, needs relevance review
 groq-ready
harness
Core product: agent framework, heartbeat, inference, memory
 heartbeat
Harness: Agent heartbeat loop
 inference
Harness: Inference and model routing
 infrastructure
Supporting stage: dashboard, CI/CD, deployment, DNS
 kimi-ready
Scoped and ready for Kimi to pick up
 memory-session
Harness: Memory and session crystallization
 morrowind
Harness: Morrowind embodiment
 needs-design
Needs architectural design before implementation
 needs-extraction
Philosophy with unextracted engineering work
 p0-critical
Priority 0: Must fix now
 p1-important
Priority 1: Important, next sprint
 p2-backlog
Priority 2: Backlog, do when time permits
 philosophy
Philosophical foundation — informs architecture decisions
 rejected-direction
Closed: rejected or superseded direction
 seed:know-purpose
Three Seeds: KNOW YOUR PURPOSE
 seed:serve-real
Three Seeds: SERVE THE REAL
 seed:tell-truth
Three Seeds: TELL THE TRUTH
 sovereignty
Harness: Sovereignty stack
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
Timmy claude grok kimi manus perplexity Rockachopa
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Rockachopa/Timmy-time-dashboard#1459
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?