Skip to content

Conversation

orecham
Copy link
Contributor

@orecham orecham commented Sep 17, 2025

Notes for Reviewer

Adds an end-to-end test for large data transfer over the Zenoh tunnel utilizing the ComplexDataType struct from the complex_data_types example.

Pre-Review Checklist for the PR Author

  • Add sensible notes for the reviewer
  • PR title is short, expressive and meaningful
  • Consider switching the PR to a draft (Convert to draft)
    • as draft PR, the CI will be skipped for pushes
  • Relevant issues are linked in the References section
  • Every source code file has a copyright header with SPDX-License-Identifier: Apache-2.0 OR MIT
  • Branch follows the naming format (iox2-123-introduce-posix-ipc-example)
  • Commits messages are according to this guideline
  • Tests follow the best practice for testing
  • Changelog updated in the unreleased section including API breaking changes
  • Assign PR to reviewer
  • All checks have passed (except task-list-completed)

Checklist for the PR Reviewer

  • Commits are properly organized and messages are according to the guideline
  • Unit tests have been written for new behavior
  • Public API is documented
  • PR title describes the changes

Post-review Checklist for the PR Author

  • All open points are addressed and tracked via issues

References

Closes #888

@orecham orecham force-pushed the iox2-888-fix-large-data-transfer-in-zenoh-tunnel branch from ec0c2bf to 493e4c9 Compare September 17, 2025 07:14
Copy link

codecov bot commented Sep 17, 2025

Codecov Report

❌ Patch coverage is 0% with 206 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.04%. Comparing base (01b372d) to head (6aff936).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
...tunnels/end-to-end-testing/ping-pong/src/pinger.rs 0.00% 98 Missing ⚠️
...tunnels/end-to-end-testing/ping-pong/src/ponger.rs 0.00% 68 Missing ⚠️
...2-tunnels/end-to-end-testing/common/src/payload.rs 0.00% 22 Missing ⚠️
iceoryx2-bb/container/src/queue.rs 0.00% 10 Missing ⚠️
...2-tunnels/end-to-end-testing/common/src/testing.rs 0.00% 6 Missing ⚠️
...oryx2-tunnels/end-to-end-testing/common/src/cli.rs 0.00% 2 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1056      +/-   ##
==========================================
- Coverage   80.21%   80.04%   -0.17%     
==========================================
  Files         258      261       +3     
  Lines       32252    32317      +65     
==========================================
- Hits        25870    25868       -2     
- Misses       6382     6449      +67     
Files with missing lines Coverage Δ
...oryx2-tunnels/end-to-end-testing/common/src/cli.rs 0.00% <0.00%> (ø)
...2-tunnels/end-to-end-testing/common/src/testing.rs 0.00% <0.00%> (ø)
iceoryx2-bb/container/src/queue.rs 91.44% <0.00%> (-3.54%) ⬇️
...2-tunnels/end-to-end-testing/common/src/payload.rs 0.00% <0.00%> (ø)
...tunnels/end-to-end-testing/ping-pong/src/ponger.rs 0.00% <0.00%> (ø)
...tunnels/end-to-end-testing/ping-pong/src/pinger.rs 0.00% <0.00%> (ø)

... and 3 files with indirect coverage changes

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

@orecham orecham force-pushed the iox2-888-fix-large-data-transfer-in-zenoh-tunnel branch 2 times, most recently from 341d4cf to 6aff936 Compare September 17, 2025 10:18
@orecham orecham self-assigned this Sep 17, 2025
@orecham orecham force-pushed the iox2-888-fix-large-data-transfer-in-zenoh-tunnel branch 2 times, most recently from 846711b to 2169c82 Compare September 17, 2025 17:27
@orecham orecham force-pushed the iox2-888-fix-large-data-transfer-in-zenoh-tunnel branch from 2169c82 to 41feb3e Compare September 17, 2025 17:32
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.

Sending Large Data with Zenoh-Tunnel fails
1 participant