Log Support in Attribute Processor (1/2)

[keitwb]

This complements the trace span support already present.

It shares the same config struct to maintain backwards-compatibility with spans in the cleanest way possible.

[codecov]

Codecov Report

Merging #1783 into master will decrease coverage by 0.00%.
The diff coverage is 88.40%.

@@            Coverage Diff             @@
##           master    #1783      +/-   ##
==========================================
- Coverage   91.29%   91.29%   -0.01%     
==========================================
  Files         273      275       +2     
  Lines       16365    16431      +66     
==========================================
+ Hits        14940    15000      +60     
- Misses        998     1003       +5     
- Partials      427      428       +1     

Impacted Files	Coverage Δ
internal/data/testdata/log.go	98.13% <0.00%> (-1.87%)	⬇️
internal/processor/filterlog/filterlog.go	92.30% <92.30%> (ø)
internal/processor/filterconfig/config.go	100.00% <100.00%> (ø)
internal/processor/filterspan/filterspan.go	96.66% <100.00%> (+3.33%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a1ac90b...7fb2147. Read the comment docs.

[bogdandrutu]

Please consider to split the PR into smaller and easier to review PRs. It is ok if for a small period of time some code is not used.

[tigrannajaryan]

Please consider to split the PR into smaller and easier to review PRs. It is ok if for a small period of time some code is not used.

@keitwb is this ready for review or you plan to split the PR?

[keitwb]

Please consider to split the PR into smaller and easier to review PRs. It is ok if for a small period of time some code is not used.

@keitwb is this ready for review or you plan to split the PR?

The stuff in internal/processor can be reviewed now since that will be the dependency of the stuff in the actual processor. I will split all of the processor proper code out to another PR whose commit will be based on the internal/processor code in this PR.

[tigrannajaryan]

@keitwb internal/processor changes look good to me. Please update the PR.

[tigrannajaryan]

I will split all of the processor proper code out to another PR whose commit will be based on the internal/processor code in this PR.

@keitwb is this still the intent?

[tigrannajaryan]

Please fix the build.

[tigrannajaryan]

@keitwb The build is still failing.

[tigrannajaryan]

LGTM