Hey folks simple smile Regarding ShellTask wouldn t it be us

Question

Hey folks simple smile Regarding ShellTask wouldn t it be useful to return the logs generated by the shell command as part of the FAIL message when the task fails Maybe we can add a parameter that keeps the current behaviour as a default to avoid breaking changes I would find it very useful for return useful details when a DbtShellTask fails What do you think

Jim Crist-Harif · Answer

Can you expand on this a bit What would you want to do with access to the logs in the task vs streaming them to cloud as is as added improved in <https github com PrefectHQ prefect pull 4322>

ale · Answer

My current use case is the ability to send those logs as part of a Slack notification

Jim Crist-Harif · Answer

Ah gotcha

ale · Answer

Does it make sense to you Jim

Jim Crist-Harif · Answer

The current `ShellTask` is kind of becoming bloated with options how we expose this may require some thought I do agree that the use case makes sense though

Jim Crist-Harif · Answer

I ll open an issue

Jim Crist-Harif · Answer

Actually is that not done already if you pass `return all=True`

Jim Crist-Harif · Answer

Zanie · Answer

It should be attached to the state yeah

Zanie · Answer

If you want it to be a part of the state message I think this is a good case for a state handler

ale · Answer

Yes it is Sorry guys I m still on `0 13 13` where this is not available Hope to upgrade to the latest very soon Thanks a lot for the guidance simple smile