Page MenuHomeElementl

[dag-star] initial starlette app for dagit

Authored by alangenfeld on Jul 9 2021, 7:34 PM.
Referenced Files
F2439253: D8807.id42301.diff
Tue, Aug 16, 7:29 PM
Tue, Aug 16, 7:29 PM
Unknown Object (File)
Mon, Aug 15, 8:06 AM
Unknown Object (File)
Sat, Aug 13, 9:12 PM
Unknown Object (File)
Fri, Aug 12, 10:22 PM
Unknown Object (File)
Fri, Aug 12, 11:46 AM
Unknown Object (File)
Wed, Aug 10, 11:53 AM
Unknown Object (File)
Fri, Aug 5, 11:54 AM



Our existing Flask + graphql-ws + gevent implementation of dagit has a number of issues

I did an exploration of starlette in D8040 which was promising. I believe we can move to this modern ASGI framework which gives us greater control and testability. This will be key in both resolving current issues and evolving dagit more effectively moving forward.

This first diff gets basic scaffolding in place and the dagit_info endpoint. Subsequent diffs will add other endpoints until we have parity with the flask app. After some further experimentation with it once working we can figure out a cut over strategy.

Test Plan

added test

Diff Detail

R1 dagster
Lint Not Applicable
Tests Not Applicable

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Jul 9 2021, 8:01 PM
Harbormaster failed remote builds in B33509: Diff 41350!
This revision is now accepted and ready to land.Jul 12 2021, 5:13 PM