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
Unknown Object (File)
Mon, Sep 9, 2:44 PM
Unknown Object (File)
Tue, Sep 3, 6:47 AM
Unknown Object (File)
Sun, Sep 1, 6:29 PM
Unknown Object (File)
Wed, Aug 28, 6:46 PM
Unknown Object (File)
Fri, Aug 23, 7:54 AM
Unknown Object (File)
Aug 19 2024, 7:25 AM
Unknown Object (File)
Jul 26 2024, 3:26 AM
Unknown Object (File)
Jul 21 2024, 9:22 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
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.)