Page MenuHomePhabricator

Reword daemon configuration issue in response to user issues
ClosedPublic

Authored by epriestley on Sep 5 2014, 2:19 AM.
Tags
None
Referenced Files
F14055227: D10421.diff
Sat, Nov 16, 9:36 AM
F14041917: D10421.diff
Mon, Nov 11, 10:47 PM
F14028279: D10421.diff
Fri, Nov 8, 12:42 PM
F14022533: D10421.diff
Wed, Nov 6, 6:05 PM
F14003170: D10421.id25083.diff
Sat, Oct 26, 3:03 AM
F14002650: D10421.diff
Fri, Oct 25, 8:56 PM
F13993123: D10421.id.diff
Tue, Oct 22, 8:49 PM
Unknown Object (File)
Sep 23 2024, 11:27 AM
Subscribers
Tokens
"Love" token, awarded by hach-que.

Details

Summary

Ref T5968. Issues we've seen from users include:

  • Concern about severity ("... Need Restarting").
    • Reduce severity of explanatory text ("Different Config", "not severe").
    • Explain consequences in more detail.
    • In D10420, make "Ignore" easier to find.
    • Scope language for the multi-machine case ("at least one daemon").
  • Confusion about why daemons need restarting.
    • Unbury the lede ("Daemons and Web Have Different Config").
    • Make it clear that the root cause is a different checksum by showing the checksum. (This just hammers home that we're comparing checksums and this issue is about config checksums and we're not making it up, the checksums probably aren't that useful on their own.)
  • Difficulty understanding how to proceed when restarting does not resolve the issue:
    • Call out steps to take on the daemon console explicitly.
    • Walk through troubleshooting PHABRICATOR_ENV.
    • Walk through troubleshooting multiple local.json.
Test Plan

Screen_Shot_2014-09-04_at_7.17.18_PM.png (1×1 px, 257 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Reword daemon configuration issue in response to user issues.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.

I predict a new problem: tl;dr :)

Nice patch though...!

This revision is now accepted and ready to land.Sep 5 2014, 5:38 PM
epriestley updated this revision to Diff 25083.

Closed by commit rPaf842600ce79 (authored by @epriestley).