Can now retrieve fields from event

[odolbeau]

Linked to #103

My configuration file look like this :

input {
  gelf {
    port => 12201
    type => gelf
  }
}

output {
  influxdb {
    host => "X.X.X.X"
    port => 8086
    db => "metrics"
    user => "user"
    password => "password"
    series => "%{message}"
    columns_from_event_fields => true
    ignore_columns => ['@timestamp']
  }
}

Th influxdb output can now automatically use all fields available in event object and send it to influxdb.

[mikn]

Hi! Thank you for this patch! I'm rather much a beginner to both ruby and logstash, so it helped me do what I actually was missing from this output module. :)

Three cosmetic comments on this; Firstly you can remove the required on the "data_points" configuration variable, secondly, the use_data_points configuration variable is rather confusing, it should be called columns_from_event_fields or similar instead (I think you also forgot an @ sign when you use it) and thirdly, the ignore_data_points should be called ignore_columns.

[odolbeau]

Hi @mikn !
Thanks for reviewing. I just update the PR. :)

Tell me if you need anything else.

[gdlx]

+1
My data is already formatted in JSON, I don't need a second formatting.

[odolbeau]

Up ?

[JulienChampseix]

@odolbeau Thanks for your output fixes/improvement.
How your output should be formatted with the latest version (commit 50184ff) ?
@jordansissel any idea when this output will be merged/patched on the master branch ?

[wiibaa]

@odolbeau could you please have time to review/comment (test?) the following pull request
logstash-plugins/logstash-output-influxdb#13
that is very similar to yours with the only advantage to already exists in the correct repository.
It would help to reduce duplicated issues/PR from this "deprecated" repository and also give more confidence to the logstash team in integrating this in the plugin.
Thanks in advance.