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

introduce option to overwrite journald parser #347

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

slomo
Copy link

@slomo slomo commented Dec 9, 2019

Since journald might contain different formats of logs, there should be an option to set the parser associated with the data. I can now setup parser like journald_myapplication in scalyr.

Since journald might contain different formats of logs, there should be an option to set the parser associated with the data. I can now setup parser like `journald_myapplication` in scalyr.
@slomo
Copy link
Author

slomo commented Dec 9, 2019

I am not sure if that is aligned with your product strategy, but the feature would be supper useful. Maybe this would even make sense on ScalyrMonitor inside https://github.com/scalyr/scalyr-agent-2/blob/master/scalyr_agent/scalyr_monitor.py in order to be available to all monitors (I guess shell monitor would also profit from this change).

At least for journal it is a super important feature because in my opinion journal is more a log transport than a specific kind of log, which can contain many log formats. In my particular case some 3rd party but also my own applications are logging into journal. I use structured log, but the thirdparty applications have "regular" log formats.

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.

1 participant