https://flyte.org logo
#ask-the-community
Title
# ask-the-community
j

Jay Ganbat

06/09/2023, 5:46 PM
Hi flyte folks, when is the next flytepropeller release, would this fix can make it to the release in flytepropeller https://github.com/flyteorg/flytepropeller/pull/571
k

Kevin Su

06/09/2023, 5:53 PM
we cut a release for every commit in propeller. https://github.com/flyteorg/flytepropeller/releases/tag/v1.1.94
j

Jay Ganbat

06/09/2023, 5:54 PM
oh i forgot we are upgrading everything at once with flyte releases, currently we are trying to update to flyte 1.6. any timeline for the entire flyte components 😄
k

Kevin Su

06/11/2023, 8:47 PM
we’ll create 1.7 release next week
j

Jay Ganbat

06/22/2023, 10:08 PM
Hi @Dan Rammer (hamersaw) i am testing this fix and it looks like this release did not fix the issue https://github.com/flyteorg/flytepropeller/pull/571 i am still getting the same error
should i re-open the issue
k

Kevin Su

06/22/2023, 10:40 PM
Could you share your workflow code, want to test it again
j

Jay Ganbat

06/22/2023, 10:41 PM
i tested the same one i posted above and getting the same result 😞
this one
Copy code
from __future__ import absolute_import, division, print_function
from typing import Optional

from flytekit import task, workflow, dynamic
from flytekit.types.file import FlyteFile

@task
def hello(myfile: Optional[FlyteFile]) -> Optional[FlyteFile]:
    return None

@dynamic
def dyn_hello(myfile: Optional[FlyteFile]) -> Optional[FlyteFile]:
    return hello(myfile=myfile)


@workflow
def child_workflow(myfile: Optional[FlyteFile]) -> Optional[FlyteFile]:
    hello_output = dyn_hello(myfile=myfile)
    return hello_output
have you had the chance to test it out @Kevin Su
k

Kevin Su

06/23/2023, 6:17 PM
I found the problem, we forgot update the propeller version in flyteadmin.
going to create a pr, sorry for the late reply.
j

Jay Ganbat

06/23/2023, 6:23 PM
oh i see, yeah no worries.
im curious how they depend on the backend, Does flyteadmin installs flytepropeller in its running container? since i have another flytepropeller container running alongside flyteadmin
k

Kevin Su

06/23/2023, 6:36 PM
I just upgrade admin, and it works for me
pingsutw/flyteadmin:673e401af18b18480c629c68ff4aaeea7f506e12
j

Jay Ganbat

06/26/2023, 4:26 PM
Hi @Kevin Su thanks for creating the fix, does flyteadmin have similar process as flytepropeller like create a release for each merge to main?
d

Dan Rammer (hamersaw)

06/26/2023, 4:27 PM
Hey @Jay Ganbat, sorry was off last week. And yes, I merged Kevins PR into flyteadmin this morning (thanks Kevin!) and there should be an official release generated.
j

Jay Ganbat

06/26/2023, 4:29 PM
that is awesome, thank you very much 🙏 ill be on the look out for that since it just merged 20min ago
just confirmed it is working in my sandbox thanks
70 Views