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

Add statsd template #256

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

Conversation

SegFaultAX
Copy link
Member

Includes:

  • Monit configuration
  • Upstart configuration (with respawn)
  • Default statsd configuration
  • Add nodejs apt repository and installs nodejs package

Includes:

* Monit configuration
* Upstart configuration (with respawn)
* Default statsd configuration
* Add nodejs apt repository and installs nodejs package
@nirvdrum
Copy link
Member

Nice. I actually have a statsd template locally I was going to merge in. But I like your use of a PPA better than compiling node.

@SegFaultAX
Copy link
Member Author

At first I wasn't sure if using a PPA was the right course of action. The only other template that relies on a PPA (that I can think of) is postgresql. What do you think about adding ppas as a rubber*.yml configuration option so they can live as a separate part of the bootstrap phase, eg:

roles:
  statsd:
    ppas: [user/ppa1, user/ppa2]

With a setup phase called rubber:add_ppas dispatched from rubber:bootstrap.

Anyway, just a thought. I'll probably hack on it when I get some free time if there is any interest for such a feature.

@mattsmith
Copy link

What's the status for this? Curious if there is any plan to merge this pull-request.

@nirvdrum
Copy link
Member

It'll get there in one form or another. Sorry for the delay in all this. I put a bit of a moratorium on new templates in order to keep the number of moving parts down while adding a few other big features. I have one outstanding major bug that I hope to get to soon and then hope to clean house on PRs.

@mattsmith
Copy link

Excellent, thanks!

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.

None yet

3 participants