revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-12-22 13:30:55 +01:00
Code Issues Releases Wiki Activity

Increase phd sanity check timeout

Browse source 
Summary:
A user reported that their install (on an unusual piece of hardware) was hitting this timeout. We don't need to be quite so stingy; just use the default 30s timeout.

Also remove some kind of sentence fragment since I no longer remember what it meant and it doesn't make sense.

Test Plan: This change resolved the issue.

Reviewers: btrahan, jungejason

Reviewed By: jungejason

CC: aran, epriestley

Differential Revision: https://secure.phabricator.com/D1805
This commit is contained in:
epriestley 2012-03-07 08:52:00 -08:00
parent f8431bbfee
commit 84d2f5fcad
1 changed files with 0 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/infrastructure/daemon/control/PhabricatorDaemonControl.php
View file

@ -192,10 +192,7 @@ EOHELP
// isn't available on the production host, etc. If any of this doesn't work, // isn't available on the production host, etc. If any of this doesn't work,
// conduit will throw. // conduit will throw.
// We do this here rather than in the daemon since there's an HTTPS + curl
// + fork issue of some kind that makes
$conduit = new ConduitClient(PhabricatorEnv::getURI('/api/')); $conduit = new ConduitClient(PhabricatorEnv::getURI('/api/'));
$conduit->setTimeout(5);
$conduit->callMethodSynchronous('conduit.ping', array()); $conduit->callMethodSynchronous('conduit.ping', array());
} }