<#6991 Fix: [Run Service] Ensure GetActionData wor...
# flyte-github
a
#6991 Fix: [Run Service] Ensure GetActionData works expected Pull request opened by SharpKoi Tracking issue Closes #6974 Why are the changes needed? The
GetActionData
endpoint in Run Service currently returns inputs missing the literals and context fields, which is unexpected. This PR fixes the issue #6974 . What changes were proposed in this pull request? Key Changes: • Fix the read path in
GetActionData
to match the write path used during storage. • In
CreateRun
, store the full
task.Inputs
(which preserves both literals and context) instead of only
task.Inputs.Literals
. How was this patch tested? This PR changes how
task.Inputs
is written and read: previously,
task.Inputs
was stored and read as a
core.LiteralMap
, preserving only literals. Now, the full
task.Inputs
is stored and read directly, preserving both literals and context. To ensure backward compatibility between the old and new formats, the following wire-compatibility tests were added in `run_service_test.go`: •
task.Inputs
written → read as
task.Inputs
•
task.Inputs
written → read as
core.LiteralMap
•
core.LiteralMap
written → read as
task.Inputs
Labels • changed: For changes in existing functionality. • fixed: For any bug fixed. 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 • Fix: [Run Service] Ensure GetActionData works expected 👈 flyteorg/flyte