Propagate telemetry.resource to the internal logs

[iblancasa]

Description

Propagate telemetry.resource to the internal logs.

Link to tracking issue

Fixes #12582

[bogdandrutu]

Please add a test when used in combination with the otlp exporter for logs

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.69%. Comparing base (2ecd0b7) to head (e2c2f91).
Report is 13 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main   #12583   +/-   ##
=======================================
  Coverage   91.69%   91.69%           
=======================================
  Files         501      503    +2     
  Lines       27506    27559   +53     
=======================================
+ Hits        25221    25271   +50     
- Misses       1806     1808    +2     
- Partials      479      480    +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[iblancasa]

Please add a test when used in combination with the otlp exporter for logs

Test added.

[iblancasa]

Can you check again @bogdandrutu? Thanks!

[iblancasa]

@open-telemetry/collector-approvers can somebody take a look to this PR? Thank you

[jade-guiton-dd]

Thank you for the PR. I have some concerns related to my comment on the tracking issue however.

[jade-guiton-dd]

This looks good to me now, but I'll leave it to @bogdandrutu to evaluate whether the test he requested is no longer necessary.

[iblancasa]

This looks good to me now, but I'll leave it to @bogdandrutu to evaluate whether the test he requested is no longer necessary.

Thanks a lot for your time to review and your help.

[jade-guiton-dd]

Looking at this again, since this PR no longer affects the export of logs through OTLP, I don't think the test requested by Bogdan makes sense anymore. Approving.

[iblancasa]

@open-telemetry/collector-maintainers can somebody take a look there?

[iblancasa]

Hey, could you take a look to this PR? @bogdandrutu @sfc-gh-bdrutu

[iblancasa]

ping @bogdandrutu @sfc-gh-bdrutu

[iblancasa]

Ping @open-telemetry/collector-approvers

[iblancasa]

@bogdandrutu @sfc-gh-bdrutu ready for review

[iblancasa]

I think everything should be ok now, @bogdandrutu @sfc-gh-bdrutu

[iblancasa]

@open-telemetry/collector-approvers could somebody merge this? Thanks!

[codeboten]

Thanks for this fix @iblancasa, this makes me wonder if at some point we'd be able to switch to using a console exporter to ensure consistency in the future