In the Worker, the runtime is currently always defaulting the timeout to its own native default.
This is confusing and misleading and wrong and will cause an early exit if the run timeout is longer than the default runtime timeout.
This PR:
Removes the runtime's default entirely
Adds a default timeout option which can be passed in externally
Moves the old default to the CLI to ensure the same behaviour
Quick note as well at the engine used to pass in timeout:0 to the runtime to disable the default - but that actually just wasn't working since I moved the timeout to the plan options.
This is not well tested - I suggest we add a bunch of tests around this in #414
In the Worker, the runtime is currently always defaulting the timeout to its own native default.
This is confusing and misleading and wrong and will cause an early exit if the run timeout is longer than the default runtime timeout.
This PR:
Quick note as well at the engine used to pass in
timeout:0to the runtime to disable the default - but that actually just wasn't working since I moved the timeout to the plan options.This is not well tested - I suggest we add a bunch of tests around this in #414