search

dagster-io / dagster

An orchestration platform for the development, production, and observation of data assets.
https://dagster.io
Apache License 2.0
11.17k stars 1.4k forks source link

Add a Dagster run status for warnings. #23107

Open Auric-Manteo opened 2 months ago

Auric-Manteo commented 2 months ago

What's the use case?

The Dagster run succeeds, but I want dagit to show that one or multiple warnings occurred during the run that should be investigated. I often find myself handling warnings outside of Dagster and checking manually if these are problematic or not Currently, these runs show up as green in dagit, but they should for example be yellow to indicate that something worth investigating happened during the run. If there are many Dagster runs this helps identify the affected runs and also gives a helpful indication of potential issues in the overview in dagit.

Ideas of implementation

There are already multiple statuses for Dagster runs. One possibility would be to hard code another status for when warnings occurred during the run and show them as yellow or orange in dagit. A more flexible solution would be to allow for a custom status and a customer color for the run.

Additional information

No response

Message from the maintainers

Impacted by this issue? Give it a 👍! We factor engagement into prioritization.

garethbrickman commented 1 month ago

Relates to: https://github.com/dagster-io/dagster/issues/22964

hellendag commented 1 month ago

Adding another type for warning sounds like it's worth discussing. Re-triaging out of UX since it's not really a question of the UX but of how status is characterized.