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
F18738426: D11699.id28152.diff
Wed, Oct 1, 3:28 PM
F18738425: D11699.id28153.diff
Wed, Oct 1, 3:28 PM
F18712393: D11699.id.diff
Mon, Sep 29, 5:23 AM
F18695470: D11699.id.diff
Sat, Sep 27, 5:33 AM
F18688491: D11699.diff
Fri, Sep 26, 4:34 PM
F18600023: D11699.diff
Sep 13 2025, 8:50 AM
F18103275: D11699.id28155.diff
Aug 10 2025, 3:37 AM
F18099900: D11699.id28152.diff
Aug 9 2025, 9:19 AM
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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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.)