Page MenuHomePhabricator

When `cluster.databases` is configured, read the master connection from it
ClosedPublic

Authored by epriestley on Apr 10 2016, 3:33 AM.
Tags
None
Referenced Files
F14062893: D15668.diff
Mon, Nov 18, 3:20 PM
F14051888: D15668.id37773.diff
Fri, Nov 15, 4:49 AM
F14046090: D15668.diff
Wed, Nov 13, 6:05 PM
F14033895: D15668.diff
Sat, Nov 9, 8:37 PM
F14021037: D15668.diff
Wed, Nov 6, 4:49 AM
F14001365: D15668.id37773.diff
Fri, Oct 25, 7:02 AM
F13998563: D15668.diff
Thu, Oct 24, 9:50 AM
F13987480: D15668.id37758.diff
Mon, Oct 21, 9:43 AM
Subscribers
None

Details

Summary

Ref T4571. Ref T10759. Ref T10758. This isn't complete, but gets most of the job done:

  • When cluster.databases is set up, most things ignore mysql.host now.
  • You can bin/storage upgrade and stuff works.
  • You can browse around in the web UI and stuff works.

There's still a lot of weird tricky stuff to navigate, and this has real no advantages over configuring a single server yet (no automatic failover, etc).

Test Plan
  • Configured cluster.databases to point at my t1.micro hosts in EC2 (master + replica).
  • Ran bin/storage upgrade, got a new install setup on them properly.
  • Survived setup warnings, browsed around.
  • Switched back to local config, ran bin/storage upgrade, browsed around, went through setup checks.
  • Intentionally broke config (bad hosts, no masters) and things seemed to react reasonably well.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to When `cluster.databases` is configured, read the master connection from it.
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 10 2016, 3:40 AM
This revision was automatically updated to reflect the committed changes.