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

Provide a dedicated mechanism for parsing logger name #397

Merged
merged 1 commit into from
Mar 24, 2022

Conversation

djaglowski
Copy link
Member

Resolves #390

@codecov
Copy link

codecov bot commented Feb 17, 2022

Codecov Report

Merging #397 (d30eaf3) into main (6c3c01e) will decrease coverage by 0.4%.
The diff coverage is 63.4%.

❗ Current head d30eaf3 differs from pull request most recent head aaa5899. Consider uploading reports for the commit aaa5899 to get more accurate results

Impacted file tree graph

@@           Coverage Diff           @@
##            main    open-telemetry/opentelemetry-log-collection#397     +/-   ##
=======================================
- Coverage   75.5%   75.0%   -0.5%     
=======================================
  Files         81      85      +4     
  Lines       3852    4065    +213     
=======================================
+ Hits        2911    3052    +141     
- Misses       663     705     +42     
- Partials     278     308     +30     
Impacted Files Coverage Δ
operator/parser/severity/severity.go 60.0% <ø> (ø)
operator/parser/time/time.go 100.0% <ø> (ø)
operator/helper/scope_name.go 56.5% <56.5%> (ø)
operator/helper/parser.go 67.5% <60.0%> (-0.5%) ⬇️
operator/parser/scope/scope_name.go 75.0% <75.0%> (ø)
entry/entry.go 97.7% <100.0%> (-2.3%) ⬇️
operator/transformer/recombine/recombine.go 75.6% <0.0%> (-2.1%) ⬇️
operator/parser/csv/csv.go 92.3% <0.0%> (-0.9%) ⬇️
operator/transformer/retain/retain.go 81.0% <0.0%> (-0.5%) ⬇️
... and 8 more

@djaglowski djaglowski force-pushed the logger-name branch 2 times, most recently from 079732f to 9c59bf0 Compare February 18, 2022 01:20
@djaglowski djaglowski marked this pull request as ready for review February 18, 2022 01:50
@djaglowski djaglowski requested a review from a team February 18, 2022 01:50
docs/types/logger_name.md Outdated Show resolved Hide resolved
entry/entry.go Outdated Show resolved Hide resolved
operator/helper/parser_test.go Outdated Show resolved Hide resolved
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Mar 8, 2022

CLA Signed

The committers listed above are authorized under a signed CLA.

docs/types/scope_name.md Outdated Show resolved Hide resolved
operator/helper/parser_test.go Outdated Show resolved Hide resolved
operator/helper/testdata/scope_name/parse_from.yaml Outdated Show resolved Hide resolved
operator/helper/testdata/scope_name/preserve_to.yaml Outdated Show resolved Hide resolved
operator/parser/json/testdata/scope_name.yaml Outdated Show resolved Hide resolved
@djaglowski djaglowski force-pushed the logger-name branch 2 times, most recently from e90c940 to bb44cea Compare March 24, 2022 16:25
@djaglowski djaglowski merged commit c357827 into open-telemetry:main Mar 24, 2022
@djaglowski djaglowski deleted the logger-name branch March 24, 2022 17:34
jsirianni pushed a commit to jsirianni/opentelemetry-log-collection that referenced this pull request Mar 28, 2022
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.

Provide dedicated capability to isolate logger name
3 participants