<#7078 [V2] Add attempt and cache status> Pull req...
# flyte-github
a
#7078 [V2] Add attempt and cache status Pull request opened by machichima 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 •
main
• #6583 • [V2] Add attempt and cache status 👈 flyteorg/flyte