Skip to content

Add configurable per-node stall timeouts - #716

Open
kurt-rhee wants to merge 1 commit into
fabro-sh:mainfrom
kurt-rhee:agent/per-node-stall-timeout
Open

Add configurable per-node stall timeouts#716
kurt-rhee wants to merge 1 commit into
fabro-sh:mainfrom
kurt-rhee:agent/per-node-stall-timeout

Conversation

@kurt-rhee

Copy link
Copy Markdown

What changed

Adds configurable stall timeouts for individual workflow nodes.

  • Reads the optional stall_timeout node attribute.
  • Uses a 3-hour default when the attribute is absent.
  • Caps configured values at 72 hours.
  • Updates the watchdog timeout whenever execution enters a new node.
  • Keeps stall timeout reporting aligned with the node-specific value.

Why

Different workflow nodes can have very different expected runtimes. A single graph-wide timeout can terminate legitimately long-running nodes or leave short-running nodes waiting too long. Per-node configuration makes the watchdog behavior match each node's workload.

Validation

  • Applied cleanly to Fabro revision 77e7704e815e9d5a46ebafad6d846085b1aeb148.
  • git diff --check passed.
  • Tests were not run in the source workspace per project instructions.

@kurt-rhee
kurt-rhee marked this pull request as ready for review August 3, 2026 18:39
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.

1 participant