Skip to content

Conversation

DenisBiryukov91
Copy link
Contributor

add qos overwrite interceptor

Copy link

PR missing one of the required labels: {'enhancement', 'internal', 'breaking-change', 'bug', 'documentation', 'new feature', 'dependencies'}

Copy link

codecov bot commented Mar 12, 2025

Codecov Report

Attention: Patch coverage is 78.53659% with 44 lines in your changes missing coverage. Please review.

Project coverage is 71.15%. Comparing base (41ac4a7) to head (2102b36).
Report is 27 commits behind head on main.

Files with missing lines Patch % Lines
zenoh/src/net/routing/interceptor/qos_overwrite.rs 82.85% 30 Missing ⚠️
zenoh/src/api/publisher.rs 0.00% 8 Missing ⚠️
commons/zenoh-config/src/qos.rs 68.42% 6 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1825      +/-   ##
==========================================
+ Coverage   71.06%   71.15%   +0.09%     
==========================================
  Files         360      363       +3     
  Lines       65135    65947     +812     
==========================================
+ Hits        46286    46925     +639     
- Misses      18849    19022     +173     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

PR missing one of the required labels: {'new feature', 'internal', 'bug', 'breaking-change', 'documentation', 'dependencies', 'enhancement'}

@DenisBiryukov91 DenisBiryukov91 added the new feature Something new is needed label Mar 12, 2025
add config validation tests;
set flows to [egress, ingress ] if not specified;
DenisBiryukov91 and others added 2 commits March 14, 2025 15:23
Co-authored-by: Oussama Teffahi <[email protected]>
Co-authored-by: Oussama Teffahi <[email protected]>
Copy link
Contributor

@oteffahi oteffahi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe the name of the config field needs to change or be moved.
Other than that, LGTM.

@oteffahi
Copy link
Contributor

This interceptor needs to be updated with the feature added in #1850

@OlivierHecart OlivierHecart enabled auto-merge (squash) March 25, 2025 17:04
@OlivierHecart OlivierHecart merged commit 3ea794a into eclipse-zenoh:main Mar 25, 2025
19 checks passed
JEnoch added a commit to ZettaScaleLabs/rmw_zenoh that referenced this pull request Apr 3, 2025
JEnoch added a commit to ZettaScaleLabs/rmw_zenoh that referenced this pull request Apr 4, 2025
Yadunund pushed a commit to ros2/rmw_zenoh that referenced this pull request Apr 4, 2025
…ic (#591)

* Bump zenoh-c to ffa4bdd, zenoh-cpp to 868fdad and zenoh to 3f62ebc

* Apply same config changes than eclipse-zenoh/zenoh#1825

* Apply same config changes than eclipse-zenoh/zenoh#1850

* For RELIABLE+TRANSIENT_LOCAL topics, enable sample_miss_detection and recovery for end-to-end reliability
mergify bot pushed a commit to ros2/rmw_zenoh that referenced this pull request Apr 4, 2025
…ic (#591)

* Bump zenoh-c to ffa4bdd, zenoh-cpp to 868fdad and zenoh to 3f62ebc

* Apply same config changes than eclipse-zenoh/zenoh#1825

* Apply same config changes than eclipse-zenoh/zenoh#1850

* For RELIABLE+TRANSIENT_LOCAL topics, enable sample_miss_detection and recovery for end-to-end reliability

(cherry picked from commit a9ab960)
mergify bot pushed a commit to ros2/rmw_zenoh that referenced this pull request Apr 4, 2025
…ic (#591)

* Bump zenoh-c to ffa4bdd, zenoh-cpp to 868fdad and zenoh to 3f62ebc

* Apply same config changes than eclipse-zenoh/zenoh#1825

* Apply same config changes than eclipse-zenoh/zenoh#1850

* For RELIABLE+TRANSIENT_LOCAL topics, enable sample_miss_detection and recovery for end-to-end reliability

(cherry picked from commit a9ab960)
Yadunund pushed a commit to ros2/rmw_zenoh that referenced this pull request Apr 5, 2025
…ic (#591) (#593)

* Bump zenoh-c to ffa4bdd, zenoh-cpp to 868fdad and zenoh to 3f62ebc

* Apply same config changes than eclipse-zenoh/zenoh#1825

* Apply same config changes than eclipse-zenoh/zenoh#1850

* For RELIABLE+TRANSIENT_LOCAL topics, enable sample_miss_detection and recovery for end-to-end reliability

(cherry picked from commit a9ab960)

Co-authored-by: Julien Enoch <[email protected]>
Yadunund pushed a commit to ros2/rmw_zenoh that referenced this pull request Apr 5, 2025
…ic (#591) (#594)

* Bump zenoh-c to ffa4bdd, zenoh-cpp to 868fdad and zenoh to 3f62ebc

* Apply same config changes than eclipse-zenoh/zenoh#1825

* Apply same config changes than eclipse-zenoh/zenoh#1850

* For RELIABLE+TRANSIENT_LOCAL topics, enable sample_miss_detection and recovery for end-to-end reliability

(cherry picked from commit a9ab960)

Co-authored-by: Julien Enoch <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new feature Something new is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants