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
Unknown Object (File)
Thu, Dec 12, 8:32 PM
Unknown Object (File)
Wed, Nov 27, 11:26 PM
Unknown Object (File)
Nov 19 2024, 4:48 PM
Unknown Object (File)
Nov 16 2024, 9:36 AM
Unknown Object (File)
Nov 11 2024, 10:47 PM
Unknown Object (File)
Nov 8 2024, 12:42 PM
Unknown Object (File)
Nov 6 2024, 6:05 PM
Unknown Object (File)
Oct 26 2024, 3:03 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).