Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Reflect changes that introduce PID in loglines #52

Open
widhalmt opened this issue Mar 4, 2021 · 1 comment · May be fixed by #53
Open

Reflect changes that introduce PID in loglines #52

widhalmt opened this issue Mar 4, 2021 · 1 comment · May be fixed by #53

Comments

@widhalmt
Copy link
Member

widhalmt commented Mar 4, 2021

As Icinga is changing its log format in Icinga/icinga2#7872 , we have to adapt our rules to that.
@widhalmt
Copy link
Member Author

widhalmt commented Mar 4, 2021

Thanks to @dgoetz for pointing that out.

widhalmt added a commit that referenced this issue Mar 4, 2021
@widhalmt
Add a new, optional, PID match to the header.
dc5047b 
This is needed because Icinga plans on changing their log format in Icinga/icinga2#7872

Thanks to @dgeotz for pointing that possible issue out.

We already had a `process.pid` in our loglines. This change also renames the already present `process.pid` to `icinga.pid`. Time will tell if this other pid will still be available in loglines.

fixes #52
@widhalmt widhalmt linked a pull request Mar 4, 2021 that will close this issue
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add a new, optional, PID match to the header. Icinga/icinga-logstash-pipeline
1 participant
@widhalmt