Reaper TTL logic unreachable for Pending/Running tasks

[coderabbitai]

Problem

The TaskReaper.Run() method in reaper/task.go has a SQL query that excludes Pending and Running states from selection, but the subsequent switch statement still contains handlers for these states. This means the TTL logic for Pending and Running tasks will never execute.

Current Query

The query currently filters for:

state IN ? and released = ?

With states: Created, Succeeded, Failed, Canceled

Missing States

The query should also include:

• task.Pending
• task.Running

Impact

Tasks in Pending or Running states will not be reaped according to their TTL settings, potentially leading to resource accumulation.

Related

• PR: ✨ Better pod retention / Perf-scale. #897
• Comment: ✨ Better pod retention / Perf-scale. #897 (comment)
• Reported by: @coderabbitai

[konveyor-ci-bot]

This issue is currently awaiting triage.
If contributors determine this is a relevant issue, they will accept it by applying the triage/accepted label and provide further guidance.
The triage/accepted label can be added by org members.