Skip to content

feat: allow immediate TaskAction GC - #7811

Open
Rohithmatham12 wants to merge 1 commit into
flyteorg:mainfrom
Rohithmatham12:codex/flyte-immediate-taskaction-gc
Open

feat: allow immediate TaskAction GC#7811
Rohithmatham12 wants to merge 1 commit into
flyteorg:mainfrom
Rohithmatham12:codex/flyte-immediate-taskaction-gc

Conversation

@Rohithmatham12

Copy link
Copy Markdown

Summary

  • allow executor GC to treat gc.maxTTL <= 0 as immediate deletion of terminal TaskActions on the next GC cycle
  • remove the setup-time validation that rejected maxTTL <= 0 when GC is enabled
  • update config/help text and add both envtest coverage and a pure unit test for the TTL decision table

Fixes #7008.

Testing

  • GOCACHE=/private/tmp/flyte-go-cache GOMODCACHE=/private/tmp/flyte-go-mod-cache go test ./executor/pkg/controller -run TestShouldDeleteTerminalTaskAction

Note: full go test ./executor/pkg/controller could not run locally because this machine is missing envtest binaries (/usr/local/kubebuilder/bin/etcd). The focused pure unit test above passed; the added Ginkgo spec should run in CI where envtest is available.

@github-actions github-actions Bot added the flyte2 label Aug 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support immediate GC cleanup when MaxTTL <= 0

1 participant