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
F17809848: D15668.diff
Fri, Jul 25, 5:08 PM
F17770284: D15668.id37773.diff
Wed, Jul 23, 9:26 AM
F17752209: D15668.diff
Tue, Jul 22, 6:41 AM
Unknown Object (File)
Jun 16 2025, 6:45 PM
Unknown Object (File)
Jun 16 2025, 5:56 AM
Unknown Object (File)
May 27 2025, 11:01 AM
Unknown Object (File)
May 5 2025, 10:28 PM
Unknown Object (File)
Apr 30 2025, 1:28 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.