Hi flyte folks, when is the next flytepropeller re...
# ask-the-community
j
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
we cut a release for every commit in propeller. https://github.com/flyteorg/flytepropeller/releases/tag/v1.1.94
j
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
we’ll create 1.7 release next week
j
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
Could you share your workflow code, want to test it again
j
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
I found the problem, we forgot update the propeller version in flyteadmin.
going to create a pr, sorry for the late reply.
j
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
I just upgrade admin, and it works for me
pingsutw/flyteadmin:673e401af18b18480c629c68ff4aaeea7f506e12
j
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
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
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
148 Views