Closed mahadzaryab1 closed 1 week ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 96.78%. Comparing base (
dd73e8a
) to head (759bb42
). Report is 3 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
in v2 we are using upstream Kafka exporter and receiver, I don't think there's anything that we need to fix.
in v2 we are using upstream Kafka exporter and receiver, I don't think there's anything that we need to fix.
Ah I see. I didn't realize that these configs are only used by v1. Is there anything that needs to be done for Kafka as there is a checkbox for it on https://github.com/jaegertracing/jaeger/issues/5229? If not, I can close out this PR and the corresponding issue.
no, nothing to do for Kafka in v2
Closed as per discussion above
Which problem is this PR solving?
Description of the changes
How was this change tested?
Checklist
jaeger
:make lint test
jaeger-ui
:yarn lint
andyarn test