Page MenuHomeElementl

[docs] sensor concept include run status sensor
ClosedPublic

Authored by yuhan on Jul 22 2021, 6:25 PM.

Details

Summary

depends on D9015

Test Plan

yarn dev

Screen Shot 2021-07-22 at 11.24.31 AM.png (2×1 px, 536 KB)

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Jul 22 2021, 6:51 PM
Harbormaster failed remote builds in B34082: Diff 42110!
yuhan requested review of this revision.Jul 22 2021, 7:53 PM

Since pipeline failure sensors are the most common case, would it make sense to put them above the run status sensor? Alternatively, would it make sense to nest them inside?

Also, probably worth linking to hear from the hooks page.

Hmm maybe I misread and the pipeline failure sensor was already nested under the run status sensor? Accepting!

This revision is now accepted and ready to land.Jul 27 2021, 1:08 AM

yup pipeline failure sensor is already nested under the run status sensor and solid hooks are still linked - i didn't update the failure sensor section in this diff.