I know resource based overrides work with `map_tas...
# flyte-support
n
I know resource based overrides work with
map_task
but the newer feature of overriding
container_image
doesn't seem to work with map_tasks. Here's a minimal example where its still using the default container image and not the overwritten one. It does work with regular tasks, just not map_task?
Copy code
@task
def my_mappable_task(a: int) -> typing.Optional[str]:
    return str(a)

@workflow
def my_map_wf(x: typing.List[int]) -> typing.List[typing.Optional[str]]:
    return map_task(
        my_mappable_task,
        concurrency=10,
        min_success_ratio=0.75,
    )(a=x).with_overrides(container_image="random:image")
cc @glamorous-carpet-83516 since he added the original feature https://github.com/flyteorg/flytekit/commit/eafcc820303367749e63edc62190b9153fd6be5e
👀 1
🙏 1
we are running 1.9.1 flyte stack if that matters
c
🙌 1
🙏 1
g
Thank you for opening an issue. @numerous-hamburger-7178 I’ll create a PR later today
🙏 1
l
Thanks for giving this your attention, Kevin!
g
🎉 1
👏 1
👀 1
c
Looking forward to using this feature as soon as it gets to a release, thanks for the quick turnaround Kevin!
g
@calm-zoo-68637 could you help me test it on your side?
Copy code
pip install git+<https://github.com/flyteorg/flytekit.git@7a30eb575e38a75a6311ab534ff8806304a1dd93>
c
Just tested, yes it works!
👍 1
g
nice, thank you. will merge it soon
c
Saw the change went in, that's awesome! Is it possible to remove the upper bound on
fsspec
that you introduced in this PR? We're getting a dependency conflict with another library that we are unable to resolve
g
which version of fsspec you are using
IIRC, there are some issues in fsspec 2023.10+, maybe they already fix it in the latest version. anyway, we can definitely make it less restrictive. like something !fsspec==2023.10.
c
we're actually not using fsspec at all
we're using s3fs which pins to 2023.10.0: https://github.com/fsspec/s3fs/blob/main/requirements.txt#L2