Skip to content

[V2] Add attempt and cache status#7078

Merged
machichima merged 7 commits intoflyteorg:v2from
machichima:add-attempt-and-cache-status
Mar 23, 2026
Merged

[V2] Add attempt and cache status#7078
machichima merged 7 commits intoflyteorg:v2from
machichima:add-attempt-and-cache-status

Conversation

@machichima
Copy link
Member

@machichima machichima commented Mar 23, 2026

Tracking issue

Why are the changes needed?

Attempts and Cache Status are missing, we should pass them from executor and store into DB

What changes were proposed in this pull request?

  • Add Attempts and CacheStatus field in CR status
  • Pass those fields to run service through UpdateActionStatus internal request and store them to DB
  • Update TaskAction naming to not contain the attempt count (one CR per action rather than on CR per attempt)
  • Update pod naming to include -n (retries = attempts - 1) as suffix

How was this patch tested?

  • Start sandbox locally and ensure:
    • Attempts and cache status being set correctly in the TaskAction CR status
    • Attempts and cache status being set correctly in the DB

TaskAction CR status: image

DB action table: image

Labels

Please add one or more of the following labels to categorize your PR:

  • added: For new features.
  • changed: For changes in existing functionality.
  • deprecated: For soon-to-be-removed features.
  • removed: For features being removed.
  • fixed: For any bug fixed.
  • security: In case of vulnerabilities

This is important to improve the readability of release notes.

Setup process

Screenshots

Check all the applicable boxes

  • I updated the documentation accordingly.
  • All new and existing tests passed.
  • All commits are signed-off.

Related PRs

Docs link

Signed-off-by: machichima <nary12321@gmail.com>
Signed-off-by: machichima <nary12321@gmail.com>
Signed-off-by: machichima <nary12321@gmail.com>
Signed-off-by: machichima <nary12321@gmail.com>
Signed-off-by: machichima <nary12321@gmail.com>
Signed-off-by: machichima <nary12321@gmail.com>
Signed-off-by: machichima <nary12321@gmail.com>
@github-actions github-actions bot mentioned this pull request Mar 23, 2026
3 tasks
@machichima machichima merged commit 2ac5750 into flyteorg:v2 Mar 23, 2026
13 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants