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

Update minsev LogProcessor implementation #5994

Merged
merged 4 commits into from
Aug 19, 2024

Conversation

MrAlias
Copy link
Contributor

@MrAlias MrAlias commented Aug 7, 2024

Implement the new definition of a LogProcessor.

Resolve #5967

Implement the new definition of a LogProcessor.
@MrAlias MrAlias added the Skip Changelog Allow PR to succeed without requiring an addition to the CHANGELOG label Aug 7, 2024
@MrAlias MrAlias marked this pull request as ready for review August 7, 2024 17:41
@MrAlias MrAlias requested a review from a team August 7, 2024 17:41
Copy link

codecov bot commented Aug 7, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 65.5%. Comparing base (074af34) to head (19074d7).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff          @@
##            main   #5994   +/-   ##
=====================================
  Coverage   65.5%   65.5%           
=====================================
  Files        203     203           
  Lines      12926   12926           
=====================================
  Hits        8474    8474           
  Misses      4198    4198           
  Partials     254     254           
Files Coverage Δ
processors/minsev/minsev.go 100.0% <100.0%> (ø)

@MrAlias MrAlias added this to the v1.29.0 milestone Aug 8, 2024
@dashpole dashpole requested a review from XSAM August 15, 2024 17:06
@MrAlias MrAlias merged commit 8b312bd into open-telemetry:main Aug 19, 2024
26 checks passed
@MrAlias MrAlias deleted the up-minsev branch August 19, 2024 15:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Skip Changelog Allow PR to succeed without requiring an addition to the CHANGELOG
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

minsev: Update after Processor changes
5 participants