god
god copied to clipboard
mojombo
PID dir can be specified for worker processes
Added command-line argument that allows the PID folder to be specified for the worker processes.
(Why? Even though a user can launch multiple god processes, all of them write pids to the same ~/.god/pids directory. Thus, each god process can clobber the PID files for the other god processes. Specifying a separate --managed_pid_dir for each god process ensures that they will not clobber one another)
Is there an ETA on merging this into master? My company has been running this branch since May 2014, so we're fairly certain that it works as expected.
It looks like there's a separate change in here that is sleeping. Can you remove that?
We found that the god command fails silently if it encounters an error in its configuration file. The sleep statement is a bit of a hack, but it allows the god command to verify that the workers are actually running before it declares victory.
Do you want me to move the sleep hack to a separate merge request? Or, would you rather that I remove the sleep hack entirely (and, possibly replace it w/ something more resilient)?
I definitely think it should be a separate discussion. I'd be interested in talking about the sorts of errors you're talking about, but I'd rather if it was not mixed in with this.
Alright, I removed the daemon launch verification logic. Sometime tomorrow, we can talk about the problem that it was intended to solve, and probably come up w/ a better way to address it.