Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] pyflyte run fails when ContainerTask is executed locally #3833

Closed
2 tasks done
zeryx opened this issue Jul 5, 2023 · 0 comments
Closed
2 tasks done

[BUG] pyflyte run fails when ContainerTask is executed locally #3833

zeryx opened this issue Jul 5, 2023 · 0 comments
Assignees
Labels
bug Something isn't working flytekit FlyteKit Python related issue

Comments

@zeryx
Copy link

zeryx commented Jul 5, 2023

Describe the bug

When using pyflyte run on a workflow that contains a ContainerTask, the ContainerTask and WF fails during tracing when attempting to run locally. When either passing --remote or registering a workflow with pyflyte register to a flyte cluster, no error is reported.

                                   TypeError: Error encountered while executing 'proxy_registration_wf':                            
                                      Failed to convert outputs of task 'register' at position output:                               
                                      Python value cannot be None, expected <class 'str'>/<FlyteLiteral simple:                      
                                    STRING>                                                                                          
Failed with Unknown Exception <class 'TypeError'> Reason: Error encountered while executing 'proxy_registration_wf':
  Failed to convert outputs of task 'register' at position output:
  Python value cannot be None, expected <class 'str'>/<FlyteLiteral simple: STRING>
Error encountered while executing 'proxy_registration_wf':
  Failed to convert outputs of task 'register' at position output:
  Python value cannot be None, expected <class 'str'>/<FlyteLiteral simple: STRING>

Expected behavior

either throw an error stating that ContainerTasks are not supported with local pyflyte run execution, or provide support for executing ContainerTasks locally.

Additional context to reproduce

https://gist.github.com/zeryx/b3d4cca7594de252a08ae8a02c2d5d6b

create a WF (or dynamic task) that triggers a ContainerTask
define the ContainerTask to have inputs and outputs
execute an arbitrary step within the ContainerTask (not reading or writing from pipes)
trigger the workflow with pyflyte run without --remote

Screenshots

No response

Are you sure this issue hasn't been raised already?

  • Yes

Have you read the Code of Conduct?

  • Yes
@zeryx zeryx added bug Something isn't working untriaged This issues has not yet been looked at by the Maintainers labels Jul 5, 2023
@eapolinario eapolinario added flytekit FlyteKit Python related issue and removed untriaged This issues has not yet been looked at by the Maintainers labels Jul 17, 2023
@eapolinario eapolinario self-assigned this Jul 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working flytekit FlyteKit Python related issue
Projects
None yet
Development

No branches or pull requests

2 participants