Skip to content

Commit

Permalink
🔄 synced local 'skyvern/' with remote 'skyvern/'
Browse files Browse the repository at this point in the history
<!-- ELLIPSIS_HIDDEN -->

> [!IMPORTANT]
> Improves failure reason messages in `setup_workflow_run` in `service.py` by including exception details.
>
>   - **Behavior**:
>     - Enhanced failure reason messages in `setup_workflow_run` in `service.py` to include exception details.
>     - Specifically, appends exception message to generic failure reason and `SkyvernException` specific message.
>
> <sup>This description was created by </sup>[<img alt="Ellipsis" src="https://img.shields.io/badge/Ellipsis-blue?color=175173">](https://www.ellipsis.dev?ref=Skyvern-AI%2Fskyvern-cloud&utm_source=github&utm_medium=referral)<sup> for d01f0f1bd7d0a1927c4a6793472ea5d4cf07cc33. It will automatically update as commits are pushed.</sup>

<!-- ELLIPSIS_HIDDEN -->
  • Loading branch information
wintonzheng committed Dec 15, 2024
1 parent bc57dc1 commit c57c3d3
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions skyvern/forge/sdk/workflow/service.py
Original file line number Diff line number Diff line change
Expand Up @@ -172,9 +172,9 @@ async def setup_workflow_run(
workflow_run_id=workflow_run.workflow_run_id,
)

failure_reason = "Setup up workflow failed due to an unexpected exception"
failure_reason = f"Setup workflow failed due to an unexpected exception: {str(e)}"
if isinstance(e, SkyvernException):
failure_reason = f"Setup workflow failed due to an SkyvernException({e.__class__.__name__})"
failure_reason = f"Setup workflow failed due to an SkyvernException({e.__class__.__name__}): {str(e)}"

await self.mark_workflow_run_as_failed(
workflow_run_id=workflow_run.workflow_run_id, failure_reason=failure_reason
Expand Down

0 comments on commit c57c3d3

Please sign in to comment.