HomePhorge

Don't fatal on daemon status updates from `phd`

Description

Don't fatal on daemon status updates from phd

Summary:
See D3126, T1667, T1658. Prior to D3126, phd did not use MySQL directly. Now that it does, there are at least two specific problems (see inline comment).

In the long term, we should probably break this dependency and use Conduit. However, we don't currently have access to the daemon log ID and getting it is a mess (the overseer generates it), and I think I want to rewrite how all this works at some point anyway (the daemon calls are currently completely unauthenticated, which is silly -- we should move them to an authenticated channel at some point, I think).

Test Plan: Ran phd stop with a bad MySQL config against a non-running daemon, didn't get a query error.

Reviewers: nh, vrana, btrahan

Reviewed By: btrahan

CC: aran

Maniphest Tasks: T1667, T1658

Differential Revision: https://secure.phabricator.com/D3314

Details

Provenance
epriestleyAuthored on Aug 16 2012, 2:13 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP2a815e0715da: Fix a PhutilOpaqueEnvelope issue with `bin/storage dump`
Branches
Unknown
Tags
Unknown

Event Timeline