Page MenuHomePhabricator

Add a `cluster.read-only` option
ClosedPublic

Authored by epriestley on Apr 8 2016, 10:41 PM.
Tags
None
Referenced Files
F14055915: D15662.diff
Sat, Nov 16, 4:42 PM
F14017440: D15662.id37746.diff
Mon, Nov 4, 5:13 PM
F14007987: D15662.id37755.diff
Tue, Oct 29, 1:43 PM
F14003719: D15662.id.diff
Sat, Oct 26, 9:48 AM
F13986008: D15662.id37746.diff
Mon, Oct 21, 12:03 AM
Unknown Object (File)
Oct 10 2024, 2:42 AM
Unknown Object (File)
Oct 7 2024, 1:41 AM
Unknown Object (File)
Oct 7 2024, 1:41 AM
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.