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)
Wed, Feb 1, 8:05 PM
Unknown Object (File)
Tue, Jan 31, 5:19 AM
Unknown Object (File)
Mon, Jan 23, 8:07 PM
Unknown Object (File)
Sat, Jan 21, 2:40 PM
Unknown Object (File)
Wed, Jan 18, 9:54 AM
Unknown Object (File)
Sun, Jan 15, 6:02 PM
Unknown Object (File)
Mon, Jan 9, 2:40 AM
Unknown Object (File)
Dec 26 2022, 6:26 PM
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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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