cloudfoundry-community / logsearch-for-cloudfoundry

A Logsearch addon that customises Logsearch to work with Cloud Foundry data
Apache License 2.0
49 stars 58 forks source link

Support for cloudfoundry/syslog-release log messages #204

Closed keymon closed 7 years ago

keymon commented 8 years ago

We found the usage of the metron agent to configure rsyslog a little bit of "hackery".

It makes sense to keep this magic wrapped in a specific job, and that is why it makes sense to use https://github.com/cloudfoundry/syslog-release

But the metron_agent syslog message and the syslog-release message are not equivalent.

The logsearch-for-cloudfoundry filters are designed to use the metron_agent format only, and fail when using the syslog release. Also note that the syslog-release format does not include the job index.

There are a pair of issues in the loggregator/metron_agent and syslog_release to get then using the same standard rfc5424

I open this issue to aggregate all the info. All the solutions require some coordination:

hannayurkevich commented 7 years ago

Hi @keymon,

Thanks for spotting this problem.

I think that it makes sense to do changes for Option1 first. So that logsearch-for-cloudfoundry supports all formats.

Then, if it is decided to apply Option2 or Option3 and a unified format is used for metron_agent and syslog_release, then we will change logsearch-for-cloudfoundry to use this format.

I think you should address Option2 and 3 to authors/contributors of metron_agent and syslog_release. Meanwhile I'm going to update logsearch-for-cloudfoundry to use Option1 soon.

Hanna

Infra-Red commented 7 years ago

Fixed in https://github.com/cloudfoundry-community/logsearch-for-cloudfoundry/pull/207