fix: Extracting timezone from cisco_ios logs#2994
Conversation
sbylica-splunk
left a comment
sbylica-splunk
left a comment
There was a problem hiding this comment.
Looks like there are some tests failing now
| '%Y-%m-%dT%T%z', | ||
| ) | ||
| template("${.tmp.timestamp}")); | ||
| template("${.tmp.date_only}")); |
There was a problem hiding this comment.
I think we should stay with tmp.timestamp and just add a (... %z) format, kind of like app-almost-syslog-citrix_netscaler.conf. I haven't tried it but will it work if you base parser here on the one from netscaler?
There was a problem hiding this comment.
I already tried that, but it didn’t work because syslog-ng does not recognize text-based timezones.
There was a problem hiding this comment.
Well, we still need to do something different because this change breaks other parsers. Also there was a release so you should rebase on top of a new develop branch that was created after that.
|
CLA Assistant Lite bot All contributors have signed the COC ✍️ ✅ |
digrajpu
force-pushed
the
fix/cisco_ios_timezone
branch
from
8bb74bf to
b03bbf2
Compare
2 participants
No description provided.