Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

carbon-aggregator doesn't start by default #28

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ddragosd
Copy link

I've added carbon-aggregator program to supervisord.conf file

I've added carbon-aggregator program to supervisord.conf file
@ivantopo
Copy link
Contributor

Hello @ddragosd, may I ask why would the image need to start carbon-aggregator? We certainly did not include any aggregation rules for it and carbon itself is not even exposed to the outside world.

@ddragosd
Copy link
Author

Over the time I've ended up using carbon-aggregator instead carbon-cache for feeding data b/c of the opportunity to aggregate data when needed. I understand it's a feature not everybody uses and is not so popular, but I personally found it useful in many cases. I'll leave it up to you to decide if you want to include it or not.

@ivantopo
Copy link
Contributor

Oh, so you are using this docker image not to feed it via StatsD but rather talking directly to carbon?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants