jaegertracing / jaeger-ui

Web UI for Jaeger
http://jaegertracing.io/
Apache License 2.0
1.15k stars 485 forks source link

Support uploads of .jsonl files #2461

Closed Saumya40-codes closed 1 month ago

Saumya40-codes commented 1 month ago

Which problem is this PR solving?

Fixes #2458

Description of the changes

How was this change tested?

Checklist

Saumya40-codes commented 1 month ago

@yurishkuro do we need to consider any other scenario while accepting jsonl files by default?

yurishkuro commented 1 month ago

Thanks!

yurishkuro commented 1 month ago

tests are failing, you need to rebuild the snapshot

Saumya40-codes commented 1 month ago

updated!

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 96.61%. Comparing base (3c88048) to head (3afaf9a). Report is 1 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #2461 +/- ## ======================================= Coverage 96.61% 96.61% ======================================= Files 254 254 Lines 7679 7679 Branches 1931 1982 +51 ======================================= Hits 7419 7419 Misses 260 260 ```

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