Page MenuHomePhabricator

Add a `cluster.read-only` option
ClosedPublic

Authored by epriestley on Apr 8 2016, 10:41 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Aug 8, 4:50 AM
Unknown Object (File)
Mon, Aug 8, 3:35 AM
Unknown Object (File)
Sun, Aug 7, 5:45 AM
Unknown Object (File)
Fri, Aug 5, 5:59 PM
Unknown Object (File)
Wed, Aug 3, 8:07 AM
Unknown Object (File)
Jul 13 2022, 7:45 PM
Unknown Object (File)
Jul 13 2022, 1:22 PM
Unknown Object (File)
Jun 27 2022, 10:55 PM
Subscribers
None

Details

Summary

Ref T4571. There will be a very long path beyond this, but add a basic read-only mode. You can explicitly enable this to put Phabricator in a sort of "maintenance" mode today if you're swapping databases or something.

In the long term, we'll automatically degrade into this mode if the master database is down.

Test Plan
  • Enabled read-only mode.
  • Browsed around.
  • Didn't immediately see anything that was totally 100% broken.

Most stuff is 80-90% broken right now. For example:

  • Stuff like submitting comments doesn't work, and gives you a confusing, unhelpful error.
  • None of the UI really knows that it's read-only. EditEngine stuff should all hide itself and say "you can't add new comments while an install is in read-only mode", for example, but currently does not.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Add a `cluster.read-only` option.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Apr 9 2016, 3:31 PM
This revision was automatically updated to reflect the committed changes.