Skip to content
This repository has been archived by the owner on May 25, 2022. It is now read-only.

Fix syslog severity mapping to be in line with specification #300

Merged

Conversation

sincejune
Copy link
Contributor

According to specification, syslog notice should be regarded as INFO2. Closes #297

@sincejune sincejune requested a review from a team November 9, 2021 07:57
Copy link
Member

@djaglowski djaglowski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the contribution @sincejune!

@djaglowski djaglowski merged commit 9cf3799 into open-telemetry:main Nov 9, 2021
@sincejune sincejune deleted the fix-syslog-severity-mapping branch November 10, 2021 06:26
djaglowski added a commit to djaglowski/opentelemetry-log-collection that referenced this pull request Nov 30, 2021
- `combine_with` setting to `recombine` operator, to allow for joining on custom delimiter ([PR315](open-telemetry#315))
- Issue where `force_flush_period` could cause line splitting to be skipped ([PR303](open-telemetry#303))
- Issue where `tcp_input` and `udp_input` could panic when stopping ([PR273](open-telemetry#273))
- Syslog severity mapping is now aligned with log specification ([PR300](open-telemetry#300))

- Improve error message when timezone database is not found ([PR289](open-telemetry#289))
djaglowski added a commit that referenced this pull request Nov 30, 2021
- `combine_with` setting to `recombine` operator, to allow for joining on custom delimiter ([PR315](#315))
- Issue where `force_flush_period` could cause line splitting to be skipped ([PR303](#303))
- Issue where `tcp_input` and `udp_input` could panic when stopping ([PR273](#273))
- Syslog severity mapping is now aligned with log specification ([PR300](#300))

- Improve error message when timezone database is not found ([PR289](#289))
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Review syslog severity mapping
2 participants