search

kestra-io / kestra

:zap: Workflow Automation Platform. Orchestrate & Schedule code in any language, run anywhere, 500+ plugins. Alternative to Zapier, Rundeck, Camunda, Airflow...
https://kestra.io
Apache License 2.0
11.09k stars 953 forks source link

Request to make `timeout` Flow.Pause Property Dynamic #5338

Open japerry911 opened 3 weeks ago

japerry911 commented 3 weeks ago

Feature description

Is it possible if Flow.Pause timeout property could be Dynamic? https://kestra.io/plugins/core/tasks/flows/io.kestra.plugin.core.flow.pause#timeout

My reasoning is I have Pause in a Subflow, and some flows will have shorter timeouts than others. Some flows will need to always be reviewed, while some will be fine without review and can have auto-approve/timeout hit.

Also worth noting is that we use Pause in a subflow, because we built slack/resuming integration that is reusable among many workflows.

This would be super helpful, but if not possible, we can make adjustments so Pause is in every flow instead of called by subflow. Thought it was worth requesting if possible to do.

Thank you 🙏

mgabelle commented 1 week ago

@loicmathieu Can you confirm that Duration still has parsing issue when it comes to v2 property ?

loicmathieu commented 1 week ago

@mgabelle I'm not sure but this task has special treatment inside the executor so it's not as simple as it seems.