Page MenuHomePhabricator
Feed Advanced Search

Aug 19 2016

epriestley added a comment to T11490: phabricator_worker.worker_archivetask has unkeyed query on dateModified.

This should now be fixed in HEAD of master. Thanks for the report!

Aug 19 2016, 6:54 PM · Restricted Project, Daemons
epriestley closed T11490: phabricator_worker.worker_archivetask has unkeyed query on dateModified as Resolved by committing Restricted Diffusion Commit.
Aug 19 2016, 6:53 PM · Restricted Project, Daemons
epriestley added a revision to T11490: phabricator_worker.worker_archivetask has unkeyed query on dateModified: Restricted Differential Revision.
Aug 19 2016, 6:32 PM · Restricted Project, Daemons

Aug 18 2016

eadler added a project to T11490: phabricator_worker.worker_archivetask has unkeyed query on dateModified: Restricted Project.
Aug 18 2016, 2:33 PM · Restricted Project, Daemons

Aug 17 2016

epriestley created T11490: phabricator_worker.worker_archivetask has unkeyed query on dateModified.
Aug 17 2016, 7:47 PM · Restricted Project, Daemons

Aug 14 2016

epriestley closed T11458: Make daemons less aggressive about cycling connections as Resolved.

Great! It looks like we're down from about 18K to 3K on repo001 too, so it appears that this pretty much works as expected.

Aug 14 2016, 1:09 PM · Restricted Project, Daemons

Aug 12 2016

jmeador added a comment to T11458: Make daemons less aggressive about cycling connections.

After merging with upstream, we were able to increase our taskmasters up to 32, and still not come anywhere close to the number of connections that were previously used. So far, so good on our end.

Aug 12 2016, 6:26 PM · Restricted Project, Daemons

Aug 11 2016

epriestley added a comment to T11458: Make daemons less aggressive about cycling connections.

I deployed this stuff, and I'm actually seeing more used ports now on secure001 than I did in D16389. However, I think most of these are coming from web requests and the larger count is because we have more traffic during US daytime than we did earlier in the morning. It didn't occur to me earlier that I'd be accidentally counting connections from both sources.

Aug 11 2016, 7:24 PM · Restricted Project, Daemons
epriestley added a revision to T11458: Make daemons less aggressive about cycling connections: Restricted Differential Revision.
Aug 11 2016, 4:07 PM · Restricted Project, Daemons
epriestley added a revision to T11458: Make daemons less aggressive about cycling connections: Restricted Differential Revision.
Aug 11 2016, 3:52 PM · Restricted Project, Daemons
FKrotkih added a comment to Q402: Why do my daemons keep <dying> ?.

I have the similar problem.. Sometimes web UI shows setup issue warning "Phabricator Daemons Are Not Running". Then I restart daemons. They work fine for some period (may be a week or two). And then again "Phabricator Daemons Are Not Running" is showed...

Aug 11 2016, 8:27 AM · Daemons
epriestley created T11458: Make daemons less aggressive about cycling connections.
Aug 11 2016, 3:04 AM · Restricted Project, Daemons

Aug 7 2016

eadler added a project to T3554: Implement a `bin/worker kill <criteria>` for batch-flushing tasks from the queue: Restricted Project.
Aug 7 2016, 8:13 PM · Restricted Project, Daemons

Aug 5 2016

eadler added a project to T7484: Lock is not locked by this process: Restricted Project.
Aug 5 2016, 5:23 PM · Restricted Project, Daemons

Jul 16 2016

epriestley moved T5969: Add timeouts to service calls to external mailers (was: PhabricatorMetaMTAWorker may hang indefinitely if "sendmail" hangs indefinitely) from Backlog to v3 on the Mail board.
Jul 16 2016, 2:09 PM · Mail

Jul 12 2016

epriestley added a comment to T11291: Error while executing task (phd daemon).

(Presuming that fixed things until we see evidence otherwise.)

Jul 12 2016, 11:29 PM · Daemons, Bug Report
epriestley closed T11291: Error while executing task (phd daemon) as Resolved by committing Restricted Diffusion Commit.
Jul 12 2016, 11:29 PM · Daemons, Bug Report
epriestley added a revision to T11291: Error while executing task (phd daemon): Restricted Differential Revision.
Jul 12 2016, 3:03 PM · Daemons, Bug Report
epriestley added a comment to T11291: Error while executing task (phd daemon).

Alright, that's promising at least. Thanks for testing!

Jul 12 2016, 2:57 PM · Daemons, Bug Report
edibiase added a comment to T11291: Error while executing task (phd daemon).

It's possible that we're in the wrong by falling back to set character set utf8;.

For anyone who can reproduce this, does this fail?

mysql> set character set utf8mb4;
Jul 12 2016, 2:47 PM · Daemons, Bug Report

Jul 11 2016

epriestley added a revision to T3554: Implement a `bin/worker kill <criteria>` for batch-flushing tasks from the queue: Restricted Differential Revision.
Jul 11 2016, 2:21 PM · Restricted Project, Daemons

Jul 10 2016

eadler added a comment to T11291: Error while executing task (phd daemon).

I think it would be better not to fallback at all and raise a setup warning. We'll likely have other issues in the future for not using the native driver.

Jul 10 2016, 1:31 AM · Daemons, Bug Report

Jul 9 2016

epriestley added a comment to T11291: Error while executing task (phd daemon).

(I made about six typos in the original version of that comment, so reload if you're following the steps from email.)

Jul 9 2016, 9:27 PM · Daemons, Bug Report
epriestley added a comment to T11291: Error while executing task (phd daemon).

It's possible that we're in the wrong by falling back to set character set utf8;.

Jul 9 2016, 9:23 PM · Daemons, Bug Report
chad added a comment to T11291: Error while executing task (phd daemon).

impressive

Jul 9 2016, 7:53 PM · Daemons, Bug Report
edibiase added a comment to T11291: Error while executing task (phd daemon).

@eadler, I was just able to resolve this on our installation by installing the php56w-mysqlnd package instead of php56w-mysql, which is what we had installed before.

Jul 9 2016, 6:48 PM · Daemons, Bug Report

Jul 8 2016

jsturgeon added a comment to T11291: Error while executing task (phd daemon).

I'm thinking the problem with our DB can be fixed by this:
https://mathiasbynens.be/notes/mysql-utf8mb4

Jul 8 2016, 9:49 PM · Daemons, Bug Report
jsturgeon added a comment to T11291: Error while executing task (phd daemon).

We're seeing an identical issue on our install. What version of MySQL are you using? Maybe we can work together and figure out how to get Emoji to work!

Jul 8 2016, 9:43 PM · Daemons, Bug Report
eadler added a comment to T11291: Error while executing task (phd daemon).

We're seeing an identical issue on our install. What version of MySQL are you using? Maybe we can work together and figure out how to get Emoji to work!

Jul 8 2016, 7:00 PM · Daemons, Bug Report
jsturgeon added a comment to T11291: Error while executing task (phd daemon).

it's definitely the 🎪 that's causing the issue, not the redacted bit. I think it's an issue with our DB, but ./storage status thinks it's fine.

Jul 8 2016, 4:36 PM · Daemons, Bug Report
epriestley added a comment to T11291: Error while executing task (phd daemon).

Whatever string this is:

Jul 8 2016, 4:34 PM · Daemons, Bug Report
jsturgeon added a comment to T11291: Error while executing task (phd daemon).

Here are the results of that command:

Jul 8 2016, 4:30 PM · Daemons, Bug Report
epriestley added a comment to T11291: Error while executing task (phd daemon).

(The harbormaster_object table is just a test table used by unit tests.)

Jul 8 2016, 4:29 PM · Daemons, Bug Report
epriestley added a comment to T11291: Error while executing task (phd daemon).

You could also try running these commands against MySQL:

Jul 8 2016, 4:28 PM · Daemons, Bug Report
epriestley added a comment to T11291: Error while executing task (phd daemon).

You can likely reproduce the issue in a repeatable way with this command:

Jul 8 2016, 4:20 PM · Daemons, Bug Report
jsturgeon added a comment to T11291: Error while executing task (phd daemon).

Unfortunately, the repo is confidential source, we might be on our own to debug this one. Perhaps there are issues with our database configuration @edibiase might have more info.

Jul 8 2016, 4:10 PM · Daemons, Bug Report
epriestley added a comment to T11291: Error while executing task (phd daemon).

Seems like that commit imported fine.

Jul 8 2016, 4:03 PM · Daemons, Bug Report
jsturgeon added a comment to T11291: Error while executing task (phd daemon).

./storage adjust returned no errors. Pushing to the GITTEST repo gave me the following error:

Jul 8 2016, 3:47 PM · Daemons, Bug Report
epriestley closed T11298: (Exception) Trying to update Task 2366703 (PhabricatorApplicationTransactionPublishWorker) after lease expiration! as Invalid.

I can't reproduce this. Bug reports MUST have reproduction steps.

Jul 8 2016, 3:34 PM · Daemons, Bug Report
jsturgeon added a project to T11291: Error while executing task (phd daemon): Daemons.
Jul 8 2016, 3:29 PM · Daemons, Bug Report
jsturgeon created T11298: (Exception) Trying to update Task 2366703 (PhabricatorApplicationTransactionPublishWorker) after lease expiration!.
Jul 8 2016, 3:28 PM · Daemons, Bug Report
epriestley closed T11295: Worker queue 404 page fails with invalid `$header` as Resolved by committing Restricted Diffusion Commit.
Jul 8 2016, 2:08 PM · Daemons, Bug Report
epriestley added a revision to T11295: Worker queue 404 page fails with invalid `$header`: Restricted Differential Revision.
Jul 8 2016, 11:34 AM · Daemons, Bug Report
epriestley added projects to T11295: Worker queue 404 page fails with invalid `$header`: Bug Report, Daemons.
Jul 8 2016, 11:29 AM · Daemons, Bug Report

Jul 6 2016

chad added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

It was merged into master today.

Jul 6 2016, 7:33 PM · Daemons, Bug Report
ftdysa added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

Looks like this should be available in 7.1 btw: https://wiki.php.net/rfc/async_signals#proposed_voting_choices.

Jul 6 2016, 7:12 PM · Daemons, Bug Report

Jul 5 2016

Mnkras added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

Of course! Thanks for poking around!

Jul 5 2016, 5:55 AM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

(And thanks for providing access to an environment where the issue reproduces!)

Jul 5 2016, 1:02 AM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

@Mnkras, I think I've undone everything I did and terminated all sessions on the host. You can revoke my key at your convenience.

Jul 5 2016, 1:00 AM · Daemons, Bug Report
eadler added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

I was writing my reply while you were writing the above. Just figured I'd mention it.

Jul 5 2016, 12:42 AM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

Yes, that's synchronous signal handling.

Jul 5 2016, 12:42 AM · Daemons, Bug Report
eadler added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

note that in the above we're explicitly handling signals. The signal_dispatch call must be placed inside of some activity loop.

Jul 5 2016, 12:41 AM · Daemons, Bug Report
eadler added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

function onsignal($signal) {

echo "Got SIGINT.\n";
  exit;

}

Jul 5 2016, 12:40 AM · Daemons, Bug Report
epriestley added a parent task for T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands: T9640: Make Phabricator compatible with PHP7.
Jul 5 2016, 12:39 AM · Daemons, Bug Report
epriestley renamed T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands from Daemons fail to stop and must be terminated to PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.
Jul 5 2016, 12:38 AM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

Yes, this reproduces on a local build of PHP 7.0.8. Here's what I did:

Jul 5 2016, 12:38 AM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

I'm building PHP 7.0.8 locally. In the meantime, I found these possibly similar reports:

Jul 5 2016, 12:27 AM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

Here's a smaller, self-contained test case:

Jul 5 2016, 12:15 AM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

I produced this test script to try to see about version issues via 3v4l.org, but you can't write files there. Requiring C as part of a different file is important, so I don't see a way to use that.

Jul 5 2016, 12:06 AM · Daemons, Bug Report

Jul 4 2016

epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

I put the global signal handler in run(), unresponsive.

Jul 4 2016, 11:52 PM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

I no-op'd the constructor and put the logic in run(), still unresponsive.

Jul 4 2016, 11:44 PM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

I disabled the entire actual daemon behavior, still unresponsive.

Jul 4 2016, 11:40 PM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

Removing the SIGINT handler entirely made it responsive.

Jul 4 2016, 11:34 PM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

I changed the SIGINT handler to exit immediately, still unresponsive to SIGINT:

Jul 4 2016, 11:32 PM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

This trivial script works correctly:

Jul 4 2016, 11:29 PM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

The PhutilNiceDaemon daemon also does not respond to ^C.

Jul 4 2016, 11:24 PM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

The debug daemon does not appear to respond to an external SIGTERM (via kill <pid> either).

Jul 4 2016, 11:21 PM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

With --trace, the daemon isn't doing any work, and also isn't hung (just normal queries to look at the queue).

Jul 4 2016, 11:20 PM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

I ran bin/phd stop and saw similar behavior:

Jul 4 2016, 11:17 PM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

@Mnkras gave me access to the host.

Jul 4 2016, 11:03 PM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

I can't reproduce that either:

Jul 4 2016, 10:49 PM · Daemons, Bug Report
Mnkras added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

Im really not sure, when I run restart this is what my daemon page looks like:

Jul 4 2016, 10:48 PM · Daemons, Bug Report
epriestley added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

I can't reproduce this. Here's what I did:

Jul 4 2016, 10:43 PM · Daemons, Bug Report
eadler moved T5334: View a graph showing worker load over time from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jul 4 2016, 9:09 PM · Restricted Project, Daemons, Facts
eadler moved T11037: `bin/phd status` is misleading about local / remote daemons from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jul 4 2016, 9:05 PM · Daemons, Restricted Project
eadler moved T10756: Make daemons work correctly no matter where they are or how many copies are running from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jul 4 2016, 9:00 PM · Restricted Project, Daemons
Mnkras updated the task description for T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.
Jul 4 2016, 8:33 PM · Daemons, Bug Report
Mnkras updated the task description for T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.
Jul 4 2016, 8:27 PM · Daemons, Bug Report
Mnkras created T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.
Jul 4 2016, 8:27 PM · Daemons, Bug Report

Jun 28 2016

anton.vladimirov added a comment to T11214: bin/phd status lists long dead daemons.

eventually the dead ones appear to fall off the edge, but in the meantime UI doesn't give me a warning that my daemons are not actually running.

Jun 28 2016, 1:56 AM · Daemons, Bug Report
anton.vladimirov added a comment to T11214: bin/phd status lists long dead daemons.

Ok, I just installed a new instance of the server in another VM.
Steps to reproduce, as far as I can tell, are:

Jun 28 2016, 1:54 AM · Daemons, Bug Report

Jun 25 2016

epriestley closed T11214: bin/phd status lists long dead daemons as Invalid.

Bug reports MUST include reproduction steps that allow us to reproduce the issue locally. Without this information, we can not move forward.

Jun 25 2016, 2:15 PM · Daemons, Bug Report
joshuaspence updated the task description for T11214: bin/phd status lists long dead daemons.
Jun 25 2016, 5:46 AM · Daemons, Bug Report
joshuaspence added a project to T11214: bin/phd status lists long dead daemons: Daemons.
Jun 25 2016, 5:45 AM · Daemons, Bug Report

Jun 13 2016

epriestley added a comment to T10811: `phd-daemon` crash after automatic restart following config change.

There's a very small possibility that D16110 fixed a race condition which was causing this, but that seems like quite a stretch.

Jun 13 2016, 8:56 PM · Daemons, Bug Report
epriestley added a revision to T10811: `phd-daemon` crash after automatic restart following config change: Restricted Differential Revision.
Jun 13 2016, 8:49 PM · Daemons, Bug Report
aubort added a project to Q419: Slow imports of commit and how to scale up: Daemons.
Jun 13 2016, 1:56 PM · Daemons

Jun 6 2016

chad updated the task description for T10811: `phd-daemon` crash after automatic restart following config change.
Jun 6 2016, 1:37 PM · Daemons, Bug Report
epriestley added a revision to T10811: `phd-daemon` crash after automatic restart following config change: Restricted Differential Revision.
Jun 6 2016, 1:35 PM · Daemons, Bug Report
epriestley added a revision to T10811: `phd-daemon` crash after automatic restart following config change: Restricted Differential Revision.
Jun 6 2016, 1:34 PM · Daemons, Bug Report

Jun 1 2016

eadler moved T10756: Make daemons work correctly no matter where they are or how many copies are running from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jun 1 2016, 10:45 PM · Restricted Project, Daemons
eadler moved T11037: `bin/phd status` is misleading about local / remote daemons from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jun 1 2016, 10:37 PM · Daemons, Restricted Project

May 27 2016

epriestley added a comment to Q402: Why do my daemons keep <dying> ?.

You can email me directly at epriestley@phacility.com, or create a private thread in Conpherence on this install via the icon in the header.

May 27 2016, 1:08 PM · Daemons
alawson added a comment to Q402: Why do my daemons keep <dying> ?.

I worked on this problem for a couple hours last night, tailing log files
and checking various security features. This is our development server, the
lead Dev and I have complete control over it. I also had this problem on
another machine which is setup very similarly (CloudLinux, but locked done
a ton as its our hosting server), but on that machine I did not put the
Phabricator installation in a cPanel account. Had the same problem there,
so I don't think it's the security / CageFS getting in the way. Something
is timing the script out or killing it, and it's not the Kernel.

May 27 2016, 1:07 PM · Daemons

May 26 2016

epriestley added a comment to T10811: `phd-daemon` crash after automatic restart following config change.

D15977 improves some behaviors slightly and adds more extensive logging for signal handling. I don't think I caught anything with real meat to it.

May 26 2016, 3:23 AM · Daemons, Bug Report
epriestley added a revision to T10811: `phd-daemon` crash after automatic restart following config change: Restricted Differential Revision.
May 26 2016, 3:19 AM · Daemons, Bug Report
epriestley added a comment to T10811: `phd-daemon` crash after automatic restart following config change.

D15975 should fix the (Exception) No such daemon "39084:6ra7uz"! error. I'm not sure if it will do anything for stability or not, but it should reduce noise. I didn't uncover anything interesting in resolving it (it wasn't really even a race, just a fairly straightforward set of state management bugs).

May 26 2016, 1:17 AM · Daemons, Bug Report
epriestley added a revision to T10811: `phd-daemon` crash after automatic restart following config change: Restricted Differential Revision.
May 26 2016, 1:13 AM · Daemons, Bug Report
alawson added a comment to Q402: Why do my daemons keep <dying> ?.

Sure thing. I left the office shortly after posting on Ponder (hence the
answer delay).

May 26 2016, 12:54 AM · Daemons
epriestley added a comment to Q402: Why do my daemons keep <dying> ?.

This is almost certainly not a resource problem, but I have no idea what it is.

May 26 2016, 12:36 AM · Daemons