#483 Populate TaskNodeMetadata on CreateNodeExecutionModel to correctly display fast cache
Pull request opened by
hamersaw
TL;DR
Current the
CacheStatus field on node executions is only set when
updating an existing model. In the case of fast cache work, where only a single node execution event will be sent for a node (ie. cache hit / succeeded), this information is never set and as a result the "cache hit" icon is never displayed in the UI. This PR sets CacheStatus if it exists on the first node execution event.
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
flyteorg/flyte#2886
Follow-up issue
NA
flyteorg/flyteadmin
GitHub Actions: Goreleaser
GitHub Actions: Build & Push Flyteadmin Image
GitHub Actions: Build & Push Flytescheduler Image
GitHub Actions: Bump Version
✅ 6 other checks have passed
6/10 successful checks