Page MenuHomePhabricator

Make the "daemons and web have different config" warning more specific
ClosedPublic

Authored by epriestley on Feb 5 2015, 9:02 PM.
Tags
None
Referenced Files
F14034979: D11699.diff
Sun, Nov 10, 3:18 AM
F14022516: D11699.diff
Wed, Nov 6, 6:00 PM
F14008306: D11699.id28153.diff
Tue, Oct 29, 7:18 PM
F13970979: D11699.id28152.diff
Thu, Oct 17, 10:51 AM
Unknown Object (File)
Oct 2 2024, 12:55 AM
Unknown Object (File)
Oct 1 2024, 12:18 AM
Unknown Object (File)
Oct 1 2024, 12:18 AM
Unknown Object (File)
Sep 21 2024, 9:25 PM
Subscribers

Details

Summary

I'm hitting this in the cluster and couldn't figure it out after staring at it for a couple minutes. Produce a better error.

This dumps a hash of each configuration key value which is set to a non-default value into the daemon log. This is much more compact than the full config, and doesn't spread secrets around, so it seems like a good balance between providing information and going crazy with it.

Test Plan

Screen_Shot_2015-02-05_at_1.00.06_PM.png (1×1 px, 357 KB)

Diff Detail

Repository
rP Phabricator
Branch
envdiff
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 4362
Build 4375: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley retitled this revision from to Make the "daemons and web have different config" warning more specific.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Feb 5 2015, 9:57 PM
This revision was automatically updated to reflect the committed changes.

Here was the culprit:

The configuration key "phabricator.env" is set in both configurations, but set to different values.

(In fairness to my sleuthing skills, we don't set that one in SiteConfig.)