Skip to content

Add tracing for colibri requests - #1301

Merged
bgrozev merged 3 commits into
jitsi:masterfrom
JulianGCalderon:tracing-jibri
Aug 18, 2026
Merged

Add tracing for colibri requests#1301
bgrozev merged 3 commits into
jitsi:masterfrom
JulianGCalderon:tracing-jibri

Conversation

@JulianGCalderon

@JulianGCalderon JulianGCalderon commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

This PR adds tracing for the handling of colibri requests.

Related to the tracing backend GSoC project: https://summerofcode.withgoogle.com/programs/2026/projects/GseggmSv

@jitsi-jenkins

Copy link
Copy Markdown

Hi, thanks for your contribution!
If you haven't already done so, could you please make sure you sign our CLA (https://jitsi.org/icla for individuals and https://jitsi.org/ccla for corporations)? We would unfortunately be unable to merge your patch unless we have that piece :(.

@JulianGCalderon
JulianGCalderon marked this pull request as ready for review August 18, 2026 09:15
@bgrozev

bgrozev commented Aug 18, 2026

Copy link
Copy Markdown
Member

jenkins add to whitelist

jitsi-ci Bot pushed a commit to jitsi/jitsi-pr-tests-pages that referenced this pull request Aug 18, 2026
@codecov

codecov Bot commented Aug 18, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 7.40741% with 25 lines in your changes missing coverage. Please review.
✅ Project coverage is 51.40%. Comparing base (b64b530) to head (e393814).
⚠️ Report is 3 commits behind head on master.

Files with missing lines Patch % Lines
...main/java/org/jitsi/jicofo/jibri/JibriSession.java 5.55% 16 Missing and 1 partial ⚠️
...rc/main/kotlin/org/jitsi/jicofo/jibri/BaseJibri.kt 11.11% 8 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1301      +/-   ##
==========================================
+ Coverage   50.47%   51.40%   +0.92%     
==========================================
  Files         131      132       +1     
  Lines        8251     8445     +194     
  Branches     1261     1289      +28     
==========================================
+ Hits         4165     4341     +176     
- Misses       3493     3494       +1     
- Partials      593      610      +17     
Files with missing lines Coverage Δ
...rc/main/kotlin/org/jitsi/jicofo/jibri/BaseJibri.kt 8.06% <11.11%> (+0.65%) ⬆️
...main/java/org/jitsi/jicofo/jibri/JibriSession.java 39.29% <5.55%> (-2.55%) ⬇️

... and 13 files with indirect coverage changes


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 076a4c7...e393814. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@bgrozev
bgrozev merged commit e98ec08 into jitsi:master Aug 18, 2026
4 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants