Enables codec entries for outputs in the ingestor_syslog job.

[jvikes11]

When setting the property ingestor_syslog.ouputs with additional logstash outputs the codec field was being set to a string. According to the documentation the codec field value for any available output is an object and not a string or integer.

https://www.elastic.co/guide/en/logstash/current/codec-plugins.html

[CLAassistant]

All committers have signed the CLA.

[bonzofenix]

@Infra-Red can you take a look at this? Thanks!

[Infra-Red]

Hi @jvikes11 @bonzofenix ! Thanks for the PR! I have few questions before attempting to merge.

1. Since codec plugin could be applied to both input and output plugins I prefer to have the same option added to input plugins.

2. Codec plugin may have additional nested options, please see the avro plugin usage example, but with the current implementation they will be just skipped.

Could you please take a look at these issues?

[bonzofenix]

what do you think about just replacing in the original implementation:
<%= k %> => <%= v.inspect %>
with just:
<%= k %> => <%= v %>

This would be a breaking change for anyone using this property but it would allow people to apply any nested config and #inspect would not break the => notation.

[bonzofenix]

the same could apply for inputs:

logsearch-boshrelease/jobs/ingestor_syslog/templates/config/input_and_output.conf.erb

Line 40 in 799a42d

<%= k %> => <%= v.inspect %>

[axelaris]

Since it's introduces a breaking change to release, I would love to see some 👍or 👎on that from community.