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
Unknown Object (File)
Sat, Mar 23, 3:46 PM
Unknown Object (File)
Sat, Mar 23, 3:42 PM
Unknown Object (File)
Sat, Mar 23, 2:21 PM
Unknown Object (File)
Sat, Mar 23, 12:46 PM
Unknown Object (File)
Sat, Mar 23, 11:46 AM
Unknown Object (File)
Sat, Mar 23, 11:35 AM
Unknown Object (File)
Tue, Mar 19, 7:17 PM
Unknown Object (File)
Tue, Mar 19, 6:25 PM
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
Branch
readonly5
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 11627
Build 14542: Run Core Tests
Build 14541: arc lint + arc unit

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.