Add auto-reload support to logstash configuration file

[davidt99]

No description provided.

[spujadas]

Good idea. Rather than having it in the args variable, could you move the option to the LS_OPTS variable?

[davidt99]

Done

[spujadas]

Cheers.

[spujadas]

Sorry, had to revert this one due to Logstash crashing (as reported in #32) when config files contain conditionals (which is the case in the default image), see elastic/logstash#5014, elastic/logstash#4968 — my mistake, didn't test the patch properly.

As this is supposed to have been fixed in Logstash 2.3.1, which was released a couple of hours ago, I'll upgrade the image and attempt to re-add the --auto-reload option (and test it properly!).

[davidt99]

I don't experiencing this issue, but I in the other hand, I don't use filebeat and the reason why I need to change the config.
I will try to look in to that in sunday.

[spujadas]

OK, that would explain.
Anyway, I've just upgraded Logstash in the image, tested it with --auto-reload, and it's running properly, so you can get the latest image (the build is in progress on Docker Hub, should be ready in a few minutes) and it should work fine.
If you do run some tests on Sunday and run into problems with this updated image, let me know.