flyteorg/flyte

[Core feature] Add timeout to ContainerTask

Open

#6,351 opened on 2025年3月20日

GitHub で見る
 (6 comments) (0 reactions) (1 assignee)Python (3,705 stars) (378 forks)batch import
enhancementgood first issuehelp wanted

説明

Motivation: Why do you think this is important?

For regular python tasks the user can set a timeout by passing timeout in the @task decorator. I think the same should be possible for ContainerTasks.

Goal: What should the final outcome look like, ideally?

e.g. ContainerTask(..., timeout=timedelta(seconds=60) should set a 60 second timeout

Describe alternatives you've considered

I think it could be done via PodTemplates atm but that seems overkill for something like this.

Propose: Link/Inline OR Additional context

No response

Are you sure this issue hasn't been raised already?

  • Yes

Have you read the Code of Conduct?

  • Yes

コントリビューターガイド

技術スタック
pythonkubernetes
領域
backend
Issue 種別
feature
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-2 days
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
blocked
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
PythonFlyte SDK basics
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
40
調査方針
Examine how timeout is implemented for regular Python tasks in the @task decorator (likely in flytekit/core/task.py). Then implement a similar timeout parameter for ContainerTask in flytekit/core/container task.py. Ensure the timeout is propagated to the Kubernetes Pod spec. Check existing tests for timeout behavior and add corresponding tests.