Page MenuHomePhabricator
Differential D10339

Daemons - add a config check for out of date daemon environment
ClosedPublic
Actions

Authored by btrahan on Aug 22 2014, 9:29 PM.
Tags
None
Referenced Files
F19154211: D10339.id24889.diff
Thu, Dec 11, 1:52 PM
F19153405: D10339.diff
Thu, Dec 11, 9:36 AM
F19153291: D10339.id24892.diff
Thu, Dec 11, 8:49 AM
F18975122: D10339.diff
Sun, Nov 16, 3:59 AM
F18868961: D10339.id.diff
Nov 4 2025, 9:23 AM
F18790693: D10339.id24891.diff
Oct 15 2025, 5:17 PM
F18762771: D10339.id.diff
Oct 6 2025, 8:32 PM
F18756817: D10339.diff
Oct 5 2025, 1:42 PM
View All Files
Subscribers
epriestley
Korvin
Tokens
"Like" token, awarded by bluehawk.

Details

Reviewers
epriestley
Maniphest Tasks
T4881: When making a config change, add a banner to let the user know to restart daemons
Commits
Restricted Diffusion Commit
rP6f246bd35190: Daemons - add a config check for out of date daemon environment
Summary

Fixes T4881.

Test Plan

made a config change, saw the issue, restarted daemons and it went away

Diff Detail

Repository
rP Phabricator
Branch
T4881
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 2332
Build 2336: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

btrahan updated this revision to Diff 24889.Aug 22 2014, 9:29 PM
btrahan added a task: T4881: When making a config change, add a banner to let the user know to restart daemons.
btrahan retitled this revision from to Daemons - add a config check for out of date daemon environment.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
Herald added subscribers: Korvin, epriestley. · View Herald TranscriptAug 22 2014, 9:29 PM
Harbormaster completed remote builds in B2332: Diff 24889.Aug 22 2014, 9:31 PM
epriestley accepted this revision.Aug 22 2014, 9:36 PM
epriestley edited edge metadata.

Nice!

src/applications/config/check/PhabricatorSetupCheckDaemons.php
77

For clarity/consistency, missing space between $ and ./

83–84

Leftover code?

This revision is now accepted and ready to land.Aug 22 2014, 9:36 PM
epriestley added inline comments.Aug 22 2014, 9:36 PM
src/applications/config/check/PhabricatorSetupCheckDaemons.php
68

Oh, maybe target => _blank here

btrahan updated this revision to Diff 24891.Aug 22 2014, 9:47 PM
btrahan edited edge metadata.

changes as requested - thanks!

Harbormaster completed remote builds in B2334: Diff 24891.Aug 22 2014, 9:49 PM
btrahan closed this revision.Aug 22 2014, 9:49 PM
btrahan updated this revision to Diff 24892.

Closed by commit rP6f246bd35190 (authored by @btrahan).

bluehawk awarded a token.Aug 23 2014, 7:56 AM

Revision Contents
Changeset List

PathSize
resources/
sql/
autopatches/
2 lines
src/
applications/
config/
check/
39 lines
daemon/
event/
1 line
storage/
1 line
infrastructure/
env/
9 lines

Diff 24889

View Options

resources/sql/autopatches/20140822.daemonenvhash.sql

Loading...
View Options

src/applications/config/check/PhabricatorSetupCheckDaemons.php

Loading...
View Options

src/applications/daemon/event/PhabricatorDaemonEventListener.php

Loading...
View Options

src/applications/daemon/storage/PhabricatorDaemonLog.php

Loading...
View Options

src/infrastructure/env/PhabricatorEnv.php

Loading...
Phabricator · Built by Phacility