Adds default value for LC tag

[jcchavezs]

LC annotation is described as:

The value of "lc" is the component or namespace of a local span.

In this case, the value should be symfony.

Does it add any value to include it @adriancole?

[roger-zhangg]

Drafted the first PR for tracer provider.

• Provided a more general Tracer Provider interface.
• Drafted a x-ray provider I wasn't included in the initial discussions in this projects so I might have missed something.

Questions:

• Decide how to refactor BaseSegment Class
• Check on current x-ray provider's design.(Doesn't support add_span, end_span)
• In the RFC there's a capture_method function included in the provider. But currently this function is in Tracer class. Do we need to move it into our provider?

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 75.22936% with 27 lines in your changes missing coverage. Please review.

Project coverage is 95.71%. Comparing base (e14e768) to head (7efe7e4). Report is 806 commits behind head on develop.

Files	Patch %	Lines
...da_powertools/tracing/provider/xray/xray_tracer.py	69.23%	17 Missing and 3 partials :warning:
aws_lambda_powertools/tracing/tracer.py	69.56%	7 Missing :warning:

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #2342      +/-   ##
===========================================
- Coverage    96.38%   95.71%   -0.67%     
===========================================
  Files          214      218       +4     
  Lines        10030    10488     +458     
  Branches      1846     1945      +99     
===========================================
+ Hits          9667    10039     +372     
- Misses         259      332      +73     
- Partials       104      117      +13     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

[leandrodamascena]

Hey @rubenfonseca! Could you please review this PR? As we discussed offline, here are some key points to consider during the review:

1 - DD and OTEL are just placeholders/examples in the code and will be removed before merging this PR. 2 - We've updated naming conventions to use more conventional names in the methods. 3 - Regarding the example, would it be better to create a new one instead of modifying the existing example?

Thanks for your attention to these points.

[roger-zhangg]

@rubenfonseca Thanks for the super detailed feedback. I've resolved all the comments now. Please help with a final review, thanks.

[sonarqubecloud[bot]]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
8.6% Duplication on New Code

See analysis details on SonarCloud

[leandrodamascena]

We can consider this PR completed, but we will not merge it now. We'll start conversations about Powertools V3 and add this refactor when we're ready for it:

1 - We will deprecate the BaseSegment in Powertools v3. 2 - This is a potential breaking change for customers who are accessing the XRay methods directly. 3 - We can take this opportunity to standardize method names like trace instead of subsegment in our documentation and ease for customers adopt new Observability providers.

I'm approving this PR and changing it to Draft state just so I don't mess up our stats.

[leandrodamascena]

Hi @roger-zhangg! I'm closing this PR in favor of #4902! It was really hard to fix the entire merge conflict here, so I've created a new one.

Thank you for all the effort you've put in here, and I'll give you credit for this work when I release v3.