Skip to content

[Feature] Support accepting simple int for dagrun_timeout, retry_delay, sla_secs and execution_timeout #528

Description

@pankajastro

Description

PR: #512 removed the inconsistent param dagrun_timeout_sec, retry_delay_sec, sla_secs, execution_timeout_secs and execution_delta_secs. This implementation is clean but it is expected that the user will use __type__: datetime.timedelta.

The aim here is to discuss the feedback #512 (comment) to allow passing an integer as seconds and implement it. We may have some poc implementation in PR: #489 for this,

Use case/motivation

No response

Related issues

No response

Are you willing to submit a PR?

  • Yes, I am willing to submit a PR!

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions