Replace master with specific jaeger tag version in otel-collector

[hanyuancheung]

Fix #2617

• Cannot use master as the "version", instead of using pined versions.
• Replace master with specific jaeger tag version in otel-collector example.

[codecov]

Codecov Report

Merging #2622 (d9e533b) into main (a1fff3c) will decrease coverage by 0.0%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##            main   #2622     +/-   ##
=======================================
- Coverage   76.0%   75.9%   -0.1%     
=======================================
  Files        173     173             
  Lines      12293   12293             
=======================================
- Hits        9344    9342      -2     
- Misses      2702    2704      +2     
  Partials     247     247             

Impacted Files	Coverage Δ
sdk/trace/batch_span_processor.go	81.1% <0.0%> (-1.0%)	⬇️

[pellared]

The URLs are invalid

it should work now

[dmathieu]

This seems a bit brittle, and we'll have to make sure we don't go too much behind on the version used.
How about rewording the example so that it points directly to jaeger's documentation instead of duplicating it here?

[MrAlias]

@hanyuancheung can you address these unresolved issues so we can move to merging this?

• Replace master with specific jaeger tag version in otel-collector #2622 (comment)
• Replace master with specific jaeger tag version in otel-collector #2622 (comment)

[hanyuancheung]

@hanyuancheung can you address these unresolved issues so we can move to merging this?

• Replace master with specific jaeger tag version in otel-collector #2622 (comment)
• Replace master with specific jaeger tag version in otel-collector #2622 (comment)

Sure. How about updating the jaeger operator version to v1.31.0, and pointing to the jaeger's documentation via an annotation in makefile?

[MrAlias]

@dmathieu please take another look.

[pellared]

Looks OK (but I have not tested it 😬 ).

[dmathieu]

I haven't tested it either. But it looks good to me.