Migrate Nomad to FastMCP v4 beta - #7
Conversation
73ebe53 to
e2409a0
Compare
2bc494b to
2cd0781
Compare
|
Looked into the failing Demo Image check; it reproduces locally off this branch's lockfiles and the mechanism is narrow. The smoke test's A few things worth knowing from the investigation:
#11 makes both markers worker-scoped: the concurrency limit subclasses Two honest caveats: the observability smoke has never executed on this branch (the job dies before reaching it), so its first run will be new information about the migration rather than about this fix; and the in-worker delegation is covered at logic level, not with a live docket round trip. This is your migration and your call: merge the stacked PR, cherry-pick it, or just take the findings. If you think the underlying sharp edge deserves an upstream report (task-sanctioned signature markers being unconditionally resolved on direct calls, with no way to scope them to task mode), I am glad to write that up too. |
Summary
Server busytool error at Nomad's defensive queue limitConfiguration
New
tool_managersettings:task_backend_url, defaultmemory://task_min_concurrency, default10device_queue_depth, default2Changed setting:
max_pending_per_toolis now finite and non-nullable, defaulting to65,536Validation
uv run ruff check src testuv run ruff format --check src testuv run pytest- 371 passedmake -C docs htmlRelease note
This PR intentionally targets the FastMCP v4 beta. Keep it as a draft and do not merge until FastMCP v4 is officially released.