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

Add missing allowed syscalls for coreos #10605

Closed
wants to merge 1 commit into from

Conversation

amencarini
Copy link

@amencarini amencarini commented Feb 6, 2019

While testing the changes made with #9994 on our Kubernetes/CoreOS setup we were getting error while waiting for event: operation not permitted lines logged. Adding these syscalls to the whitelist allowed us to avoid the error while enjoying continued logging after a journald rotation.

cc @kvch

@amencarini amencarini requested a review from a team as a code owner February 6, 2019 09:54
@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@kvch
Copy link
Contributor

kvch commented Feb 6, 2019

Thank you! How did you test these syscalls?
I have opened a PR with more to the whitelist yesterday: #10593
I seem to have more syscalls than you. :D

@kvch
Copy link
Contributor

kvch commented Feb 6, 2019

Also, were you able to read entries after logrotation or after starting Journalbeat?

@amencarini amencarini closed this Feb 6, 2019
@amencarini
Copy link
Author

We found out the relevant syscalls by using a combination of strace and checking the C code of systemd! I've closed this PR as yours is definitely more comprehensive 😆 Just tested it and after forcing a rotation logs are still shipped 🎉

@kvch
Copy link
Contributor

kvch commented Feb 6, 2019

What systemd version do you have?

@amencarini
Copy link
Author

233

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants