feat: add sampling attributes for jaeger remote sampler

[etilite]

Which problem is this PR solving?

Resolves #3653
Resolves #691

Description of the changes

• Added option WithAttributesOn which adds jaeger.sampler.param and jaeger.sampler.type attributes for sampled spans

[linux-foundation-easycla]

The committers listed above are authorized under a signed CLA.

• ✅ login: dmathieu / name: Damien Mathieu (55879b2)

[flc1125]

cc code owner: @yurishkuro PTAL

[dmathieu]

Those attributes don't match anything defined in semantic conventions.

[yurishkuro]

I suggest writing a test in a different package like jaegerremote_test, to make sure you got the visibility of new functions / types correctly

[yurishkuro]

Thank you for this PR! Originally when I looked at this problem there was no way in the API to pass back attributes (I think I even had an option issue about that).

[etilite]

Those attributes don't match anything defined in semantic conventions.

Can we do anything about it? Those attributes provide back-compatibility with jaeger-remote-sampler and can be turned only by an option. So yeah it is kinda jaeger attributes.

Wrong assessment.

[etilite]

@dmathieu @yurishkuro If you’re okay with it, I could try a different approach - add two new options - functions that create attributes for probabilistic/ratelimiting samplers, so user can define their own attributes to solve this issue.
This solution does not interfere with the semantic convention.

[dmathieu]

Sorry, my assessment was wrong. As these are jaeger specific, they shouldn't need semantic conventions.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.8%. Comparing base (7ff03d6) to head (55879b2).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@          Coverage Diff          @@
##            main   #8073   +/-   ##
=====================================
  Coverage   81.8%   81.8%           
=====================================
  Files        194     194           
  Lines      13362   13380   +18     
=====================================
+ Hits       10936   10954   +18     
  Misses      2029    2029           
  Partials     397     397           

Files with missing lines	Coverage Δ
samplers/jaegerremote/sampler.go	92.0% <100.0%> (+1.0%)	⬆️
samplers/jaegerremote/sampler_remote.go	90.4% <100.0%> (ø)
samplers/jaegerremote/sampler_remote_options.go	98.0% <100.0%> (+<0.1%)	⬆️

🚀 New features to boost your workflow:

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

[etilite]

@yurishkuro I added new test in different package and answered on all comments. Review pls

[flc1125]

@yurishkuro I added new test in different package and answered on all comments. Review pls

We can fix the CI errors before entering the review to reduce some trivial checks. 😁

[flc1125]

@etilite please add a CHANGELOG

[flc1125]

@etilite There are currently some conflicts; please fix them.

[etilite]

@flc1125 resolved conflicts but now ci/cd failing. Probably need just to retry these jobs, can't do it myself.

[yurishkuro]

👏

@etilite do you want to make the corresponding changes in Jaeger to recognize the prefixed attributes?

[etilite]

👏

@yurishkuro I'm not familiar with Jaeger backend library so it will be nice if you can handle it