Hi flyte folks, when is the next flytepropeller re...
# flyte-support
m
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
👍 1
g
we cut a release for every commit in propeller. https://github.com/flyteorg/flytepropeller/releases/tag/v1.1.94
m
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 😄
g
we’ll create 1.7 release next week
m
Hi @hallowed-mouse-14616 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
g
Could you share your workflow code, want to test it again
m
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 @glamorous-carpet-83516
g
I found the problem, we forgot update the propeller version in flyteadmin.
going to create a pr, sorry for the late reply.
m
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
g
I just upgrade admin, and it works for me
🙏 1
pingsutw/flyteadmin:673e401af18b18480c629c68ff4aaeea7f506e12
m
Hi @glamorous-carpet-83516 thanks for creating the fix, does flyteadmin have similar process as flytepropeller like create a release for each merge to main?
h
Hey @magnificent-teacher-86590, 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.
m
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
🙌 2
148 Views