search

logstash-plugins / logstash-output-influxdb

Apache License 2.0
58 stars 77 forks source link

Update to work with influxdb 0.9 - try 2 #30

Closed timgriffiths closed 8 years ago

timgriffiths commented 8 years ago

So this captures everything that is in pull request #29 but also handle all the special "characters" the influxdb protocol wants your to treat in a special way, ie spaces and commas and equal signs.

elasticsearch-release commented 8 years ago

Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run; then say 'jenkins, test it'.

contentfree commented 8 years ago

@timgriffiths, You've got to sign the CLA. (I think you said you had in #29, but it doesn't appear so here)

timgriffiths commented 8 years ago

I really thought i did ... let me try again 

Created: 2/1/16
By: CLA Committers (committers@elasticsearch.com)
Status: SIGNED
Transaction ID: CBJCHBCAABAAEUxKCiz3OGog52g8P8osKhzO52D6mVUs
timgriffiths commented 8 years ago

Now doubly signed up

Created: 2/3/16
By: CLA Committers (committers@elasticsearch.com)
Status: SIGNED
Transaction ID: CBJCHBCAABAA6SjtIjPPAI_VAHThA0sINoFQ4njqOKNX
suyograo commented 8 years ago

Tests pass

suyog@machine:~/ws/elastic/ls_plugins/logstash-output-influxdb (pr/30)$ bundle exec rspec
Using Accessor#strict_set for specs
Run options: exclude {:redis=>true, :socket=>true, :performance=>true, :couchdb=>true, :elasticsearch=>true, :elasticsearch_secure=>true, :export_cypher=>true, :integration=>true, :windows=>true}
.........

Finished in 1.62 seconds (files took 1.37 seconds to load)
9 examples, 0 failures

Randomized with seed 25191

Also confirmed CLA