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
Importing the old repo into this one #1
Conversation
…se the new logstash-core gem.
Allow for greater flexibility around polling metrics from multiple namespaces. Fixes logstash-plugins#1
- Break the run method into smaller methods - Make some logging statements info - Merge in the dimension and metric parameters - Remove method name confusion - Fixes logstash-plugins#12 - Fixes logstash-plugins#7
Fixes logstash-plugins#9 Clean up some dependencies
@@ -0,0 +1,3 @@ | |||
source 'https://rubygems.org' | |||
ruby "1.9.3", :engine => "jruby", :engine_version => "1.7.19" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why this line? can we remove it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See this for an explanation: logstash-plugins/logstash-input-example#3
@jrgns can you add tests? Even if they are mocks should be ok |
I've added some boilerplate for tests and a single test to check that the register method works. |
Importing the old repo into this one
@jrgns thanks. I merged and bumped version to 1.1.0. Can you please add |
@suyograo Done Awesome, thanx! |
No description provided.