#392 Mark webapi task failure as retry limit exceeded Pull request opened by pingsutw TL;DR fixed https://flyte-org.slack.com/archives/C05GQ16TQLF/p1692310075236329 propeller updates the

cache.state

to PhaseSystemFailure at here. however, it sets the action to

cache.Unchanged

in

cache.ItemSyncResponse

, which won't update the item in the LRU cache. Add an error message field to

cache.state

, and use it when we return

PhaseInfoFailure

. Therefore, we can see the error message on FlyteConsole if the propeller fails to get the task status. Type ☑︎ Bug Fix ☐ Feature ☐ Plugin Are all requirements met? ☑︎ Code completed ☑︎ Smoke tested ☑︎ Unit tests added ☑︎ Code documentation added ☑︎ Any pending items have an associated Issue Complete description Tracking Issue NA Follow-up issue NA flyteorg/flyteplugins Codecov: 62.90% (-0.12%) compared to 03dc9db Codecov: 61.90% of diff hit (target 63.01%) ✅ 5 other checks have passed 5/7 successful checks