https://flyte.org logo
Title
j

jeev

02/15/2022, 6:50 PM
y

Yee

02/15/2022, 6:54 PM
just ignore pod
and run function right?
i defer to @Ketan (kumare3) - I think this is one of those cases where we should ignore correctness and err on the side of user-friendliness
j

jeev

02/15/2022, 6:56 PM
we can leave it up to the users right?
we're looking into overriding this method with the base task method haha
kinda hacky
k

Ketan (kumare3)

02/15/2022, 6:59 PM
I think we should run it
At some point we used to run user code, dont know why the exception got added again
j

jeev

02/15/2022, 7:00 PM
@Greg Gydush
can we open a PR to toss that method in pod tasks?
k

Ketan (kumare3)

02/15/2022, 7:00 PM
👍
j

jeev

02/15/2022, 7:00 PM
or maybe we can add a warning
and run it anyway?
wdyt @Greg Gydush
k

Ketan (kumare3)

02/15/2022, 7:01 PM
yes i think warning and running it would be great
👍 2
g

Greg Gydush

02/15/2022, 7:38 PM
j

jeev

02/15/2022, 8:05 PM
lgtm! thanks @Greg Gydush
y

Yee

02/15/2022, 9:49 PM
what version are you guys on?
i’ll cherry pick the pr
j

jeev

02/15/2022, 9:50 PM
we have different teams that are on different versions
but we can consolidate
ok to release to latest version
wdyt @Greg Gydush
g

Greg Gydush

02/15/2022, 9:50 PM
agreed, cherrypicking to latest would be perfect
@Eduardo Apolinario (eapolinario) +1?
👍 1