0.9.12 nodaemon backport

[jib]

We run all our carbon daemons under supervisor, and not having this patch is a tremendous performance penalty as it forces us to run in --debug mode, so we backported this fix from master.

Please consider integrating it in the 0.9.x branch so we (and others) can keep up with 0.9.x releases.

[dbeckham]

Not even a single comment on this? The Internet is filled with people trying to figure out how to deal with the carbon daemons not being able to be reasonably started and stopped using modern process control. Having a --no-daemon options fixes this entirely. Why force people to use Twisted unnecessarily?

[pcn]

+1 from me. I like this. It just needs to back out the twisted version peg so that the merge can happen.

[deniszh]

+1
Not sure if --debug in Twisted has some performance impact, but having proper cli flag is better anyway...

[dbeckham]

This is a copy/paste from my comment in #191:

I spent some time yesterday researching exactly how to properly control Graphite carbon-cache, as it sits in the v0.9.12 tag, using Upstart and have posted a working config here: https://gist.github.com/dbeckham/8057390

Using the info from carbon/conf.py, carbon/util.py, and the Internet, I've created a twistd script block with the correct options, including a --nodaemon option. The config can be tweaked to send logs to a different destination including stdout, or to run under a specific instance name.

I'll be rolling this out in our Puppet environment shortly, but If anyone has any issues with it, or suggestions for making it better, please comment on the gist.

[dcarley]

👍

[pyr]

this should go in. 🤘

[obfuscurity]

@jib Would you mind rebasing against 0.9.x and we'll see about finally merging this in?

[obfuscurity]

Closing this one since we merged in @deniszh's rebased version. @jib thanks for your help with this! ⚡