Page MenuHomeElementl

[dag-star] initial starlette app for dagit
ClosedPublic

Authored by alangenfeld on Jul 9 2021, 7:34 PM.

Details

Summary

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

Repository
R1 dagster
Branch
star (branched from master)
Lint
Lint Passed
Unit
No Test Coverage

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