Thread
#flytekit
    j

    jeev

    7 months ago
    Yee

    Yee

    7 months ago
    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

    7 months ago
    we can leave it up to the users right?
    we're looking into overriding this method with the base task method haha
    kinda hacky
    Ketan (kumare3)

    Ketan (kumare3)

    7 months ago
    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

    7 months ago
    @Greg Gydush
    can we open a PR to toss that method in pod tasks?
    Ketan (kumare3)

    Ketan (kumare3)

    7 months ago
    👍
    j

    jeev

    7 months ago
    or maybe we can add a warning
    and run it anyway?
    wdyt @Greg Gydush
    Ketan (kumare3)

    Ketan (kumare3)

    7 months ago
    yes i think warning and running it would be great
    Greg Gydush

    Greg Gydush

    7 months ago
    j

    jeev

    7 months ago
    lgtm! thanks @Greg Gydush
    Yee

    Yee

    7 months ago
    what version are you guys on?
    i’ll cherry pick the pr
    j

    jeev

    7 months ago
    we have different teams that are on different versions
    but we can consolidate
    ok to release to latest version
    wdyt @Greg Gydush
    Greg Gydush

    Greg Gydush

    7 months ago
    agreed, cherrypicking to latest would be perfect
    @Eduardo Apolinario (eapolinario) +1?