Skip to content

Commit

Permalink
wrapping flyte entity in a task node in call to flyte node constructo…
Browse files Browse the repository at this point in the history
…r, not sure if integration tests are actually running (#1422)

Signed-off-by: Yee Hing Tong <wild-endeavor@users.noreply.github.com>

Signed-off-by: Yee Hing Tong <wild-endeavor@users.noreply.github.com>
  • Loading branch information
wild-endeavor authored and eapolinario committed Feb 8, 2023
1 parent 76d607a commit 9534aeb
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
4 changes: 2 additions & 2 deletions flytekit/remote/remote.py
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@
NotificationList,
WorkflowExecutionGetDataResponse,
)
from flytekit.remote.entities import FlyteLaunchPlan, FlyteNode, FlyteTask, FlyteWorkflow
from flytekit.remote.entities import FlyteLaunchPlan, FlyteNode, FlyteTask, FlyteTaskNode, FlyteWorkflow
from flytekit.remote.executions import FlyteNodeExecution, FlyteTaskExecution, FlyteWorkflowExecution
from flytekit.remote.interface import TypedInterface
from flytekit.remote.lazy_entity import LazyEntity
Expand Down Expand Up @@ -1460,7 +1460,7 @@ def sync_execution(
upstream_nodes=[],
bindings=[],
metadata=NodeMetadata(name=""),
flyte_task=flyte_entity,
task_node=FlyteTaskNode(flyte_entity),
)
}
if len(task_node_exec) >= 1
Expand Down
1 change: 1 addition & 0 deletions tests/flytekit/integration/remote/test_remote.py
Original file line number Diff line number Diff line change
Expand Up @@ -221,6 +221,7 @@ def test_fetch_execute_task_convert_dict(flyteclient, flyte_workflows_register):
flyte_task = remote.fetch_task(name="workflows.basic.dict_str_wf.convert_to_string", version=f"v{VERSION}")
d: typing.Dict[str, str] = {"key1": "value1", "key2": "value2"}
execution = remote.execute(flyte_task, {"d": d}, wait=True)
remote.sync_execution(execution, sync_nodes=True)
assert json.loads(execution.outputs["o0"]) == {"key1": "value1", "key2": "value2"}


Expand Down

0 comments on commit 9534aeb

Please sign in to comment.