Page MenuHomePhabricator

Add an extensible "SiteSource" for configuration
ClosedPublic

Authored by epriestley on Nov 5 2014, 1:34 PM.
Tags
None
Referenced Files
F15524922: D10787.diff
Mon, Apr 21, 11:14 AM
F15519136: D10787.id25901.diff
Sat, Apr 19, 8:08 PM
F15494491: D10787.id25901.diff
Sun, Apr 13, 12:09 AM
F15483228: D10787.id25901.diff
Wed, Apr 9, 10:54 AM
F15480996: D10787.id25885.diff
Tue, Apr 8, 5:43 PM
F15480840: D10787.id.diff
Tue, Apr 8, 4:54 PM
F15477740: D10787.diff
Mon, Apr 7, 5:32 PM
F15437311: D10787.id.diff
Tue, Mar 25, 6:51 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