Page MenuHomeElementl

unbreak dagit read-only mode
ClosedPublic

Authored by dgibson on Jul 12 2021, 1:03 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, May 11, 2:09 AM
Unknown Object (File)
Sun, Apr 30, 4:07 AM
Unknown Object (File)
Apr 29 2023, 12:03 AM
Unknown Object (File)
Apr 28 2023, 6:22 AM
Unknown Object (File)
Apr 2 2023, 7:43 AM
Unknown Object (File)
Mar 31 2023, 2:35 AM
Unknown Object (File)
Mar 10 2023, 9:44 PM
Unknown Object (File)
Mar 6 2023, 1:22 AM
Subscribers
None

Details

Summary

While testing some read-only related changes it became clear that a bad refactor caused the read_only param to the dagit CLI to get thrown out :( This diff threads it through correctly.

Test Plan

BK, load dagit with --read-only and verify you can't launch pipelines

Diff Detail

Repository
R1 dagster
Branch
rmreadonly3
Lint
Lint Passed
Unit
No Test Coverage

Event Timeline

This revision is now accepted and ready to land.Jul 12 2021, 3:44 PM
This revision was landed with ongoing or failed builds.Jul 12 2021, 3:54 PM
This revision was automatically updated to reflect the committed changes.

this behavior should be under test, something in python_modules/dagit/dagit_tests/test_app.py maybe