Page MenuHomeElementl

[dagit] Surface errors on backfill mutation

Authored by dish on Nov 30 2020, 11:05 PM.



Resolves #3262.

When errors are returned from GraphQL when kicking off a backfill, we currently swallow the errors. Instead, display them more aggressively.

This isn't a great long term solution, but we also don't have really good ways of consistently surfacing stack traces in Dagit UI. It's certainly better than nothing though.

Test Plan

Create a schedule that uses an invalid mode, try to kick off a backfill for it. Verify that the error appears.

Diff Detail

R1 dagster
Lint Not Applicable
Tests Not Applicable

Event Timeline

awesome, looks good! just one small comment


If we don't have result.message, should we still show a message like "something bad happened and the backfill was not successfully launched, plz report bug to dagster"

This revision is now accepted and ready to land.Dec 1 2020, 6:35 AM

Add generic message on unknown failure, add a link to view the error