Page MenuHomePhabricator

Add an extensible "SiteSource" for configuration
ClosedPublic

Authored by epriestley on Nov 5 2014, 1:34 PM.
Tags
None
Referenced Files
F14358025: D10787.diff
Fri, Dec 20, 7:30 AM
F14352365: D10787.id25901.diff
Thu, Dec 19, 1:39 PM
F14352364: D10787.id25885.diff
Thu, Dec 19, 1:39 PM
F14352363: D10787.diff
Thu, Dec 19, 1:38 PM
Unknown Object (File)
Tue, Dec 17, 3:04 AM
Unknown Object (File)
Wed, Nov 27, 9:08 AM
Unknown Object (File)
Sat, Nov 23, 1:59 PM
Unknown Object (File)
Wed, Nov 20, 3:34 PM
Subscribers
Tokens
"Mountain of Wealth" token, awarded by hach-que."Doubloon" token, awarded by btrahan.

Details

Reviewers
btrahan
Maniphest Tasks
Restricted Maniphest Task
Commits
Restricted Diffusion Commit
rP6a5369b173d5: Add an extensible "SiteSource" for configuration
Summary

Fixes T2792. This adds a pluggable configuration layer between all the stuff on disk (local/file) and the runtime configurable stuff (database).

An install can subclass this source and:

  • For Phacility, query a remote service (like Almanac) to retrieve hostname-based configuration, allowing one install to serve multiple instances.
  • Maybe for Phacility, query a remote service (like Phlux) to retrieve sitevar-like configuration (e.g., put everything in readonly mode to deal with a maintenance issue?). Not sure if we'll do this or not. We might just nuke Phlux since Almanac is sort-of-a-superset of it for our purposes.
  • For third parties, query some other remote service if that makes config management easier. In particular, it would theoretically let you put locked config in Zookeeper or whatever else you want.
Test Plan

Added a fake source and saw it inject configuration.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable