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.

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

Syslog format by default #392

Merged
merged 1 commit into from
Dec 13, 2021
Merged

Conversation

gregshep
Copy link
Contributor

Overall Review of Changes:
Use syslog format for one line per timestamp for better external logger support (i.e. Splunk)

Issue Fixes:
By default, aide logs are multi-line without timestamp making external logger parsing complex.

Enhancements:
Each log output line has a timestamp

How has this been tested?:
Tested locally with an instance using this repo, verified output logs are syslog format.

@georgenalen georgenalen self-assigned this Nov 23, 2021
@georgenalen
Copy link
Contributor

@gregshep,
Thanks for setting up this PR. one thing that will need to be updated before we can merge this in is the DCO setup. For us to merge it will need to have all commits GPG signed and have the signed-off by added. Once those are done the DCO check will show a green check and I can then work on merging the work.

George

Use syslog format for one line per timestamp for better external logger support (i.e. Splunk)

Signed-off-by: Gregory Shepherd <gregory.shepherd@parsons.com>
@uk-bolly uk-bolly changed the base branch from devel to staging_dec21 December 13, 2021 11:24
@uk-bolly uk-bolly merged commit 511c1bb into ansible-lockdown:staging_dec21 Dec 13, 2021
@uk-bolly
Copy link
Member

Hi Greg

Thank you for following up with the sign off. I have created a new branch to stage these changes into. Hoping to get this pulled into devel over the next week.

Thanks

uk-bolly

@uk-bolly uk-bolly mentioned this pull request Dec 29, 2021
@georgenalen georgenalen mentioned this pull request Jan 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants