Skip to content

Conversation

@cijothomas
Copy link
Member

#1815 added saturation test to OTAP-OTAP scenario, but used the config with wait_for_result:true. But OTAP receiver does not have that implemented. Back-pressure tests were already not running these

This PR modified the saturation test with OTAP to use the wait-for-result:false mode until above is addressed.

@cijothomas cijothomas requested a review from a team as a code owner January 20, 2026 18:57
@codecov
Copy link

codecov bot commented Jan 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.66%. Comparing base (448d7ac) to head (f5e2c2b).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1838      +/-   ##
==========================================
- Coverage   84.66%   84.66%   -0.01%     
==========================================
  Files         499      499              
  Lines      146784   146784              
==========================================
- Hits       124279   124272       -7     
- Misses      21971    21978       +7     
  Partials      534      534              
Components Coverage Δ
otap-dataflow 86.06% <ø> (-0.01%) ⬇️
query_abstraction 80.61% <ø> (ø)
query_engine 90.52% <ø> (ø)
syslog_cef_receivers ∅ <ø> (∅)
otel-arrow-go 53.50% <ø> (ø)
quiver 90.66% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jmacd jmacd added this pull request to the merge queue Jan 20, 2026
Merged via the queue into open-telemetry:main with commit 803090a Jan 20, 2026
44 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants