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

Monitor RabbitMQ queue sizes and node memory/disk health #93

Merged
merged 2 commits into from
Dec 24, 2014

Conversation

mpalmer
Copy link
Contributor

@mpalmer mpalmer commented Dec 19, 2014

No description provided.

@@ -1,7 +1,7 @@
#!/usr/bin/env ruby
#

require File.expand_path('../../lib/riemann/tools', __FILE__)
require 'riemann/tools'
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wait, isn't riemann-rabbitmq a part of the riemann-tools gem still? We can't depend on the gem this way unless it's a separate dep, right?

@aphyr
Copy link
Collaborator

aphyr commented Dec 23, 2014

This looks mostly good, just want to be reassured about the require change and we'll be set! It's been a while since I've done ruby packaging so I don't want to accidentally break anything. :)

@mpalmer
Copy link
Contributor Author

mpalmer commented Dec 24, 2014

On a system with RubyGems activated (ie 1.9+ by default, or 1.8 with RUBYOPT=-rubygems) require is implemented in such a way that gems installed on the system will be scanned for the existence of the named file, and will activate the gem (ie add it to $LOAD_PATH) that contains the file, if one is found.

On a more concrete note, a patched riemann-rabbitmq, in a rebuilt gem, with this change is running quite happily for me on several machines now, so I'm pretty confident it won't break anything.

@aphyr
Copy link
Collaborator

aphyr commented Dec 24, 2014

Yeah, the question is more which copy of the code it refers to. The reason I require the local copy is because when you're hacking on riemann-tools proper, you want those changes to be reflected in the tools themselves. If I understand correctly, this require line will read the installed rubygems copy of riemann/tools.rb, not the local copy.

@aphyr
Copy link
Collaborator

aphyr commented Dec 24, 2014

Just gonna merge this and flip back the require line. :)

aphyr added a commit that referenced this pull request Dec 24, 2014
Monitor RabbitMQ queue sizes and node memory/disk health
@aphyr aphyr merged commit fde02b0 into riemann:master Dec 24, 2014
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