<#7096 Fix: Make flytectl hydrate pod template spe...
# flyte-github
a
#7096 Fix: Make flytectl hydrate pod template spec in task node overrides Pull request opened by fg91 Why are the changes needed? @workflow def wf(): some_task().with_overrides( pod_template=PodTemplate( primary_container_name="primary", pod_spec=client.V1PodSpec( containers=[], node_selector={"foo", "bar"} ), ) ) When running this workflow with
pyflyte run --remote
, it works. With
pyflyte package
and
flytectl register
, it fails because the code tarball bucket uri isn't templated in the pod container args: - args: - pyflyte-fast-execute - --additional-distribution - '{{ .remote_package_path }}' What changes were proposed in this pull request? The templating happens during what
flytectl register
calls "hydration". It is done correctly for tasks specs but it is just omitted for task node overrides. This PR fixes this. How was this patch tested? • Added unit test. • Tested that the example workflow above succeeds in a cluster when registered with the fixed
flytectl
. Check all the applicable boxes • I updated the documentation accordingly. • All new and existing tests passed. • All commits are signed-off. flyteorg/flyte