Skip to content

Conversation

@OlivierHecart
Copy link
Contributor

@OlivierHecart OlivierHecart commented Jan 14, 2026


🏷️ Label-Based Checklist

Based on the labels applied to this PR, please complete these additional requirements:

Labels: internal

🏠 Internal Change

This PR is marked as internal (not user-facing):

  • No API changes - Public APIs unchanged
  • No behavior changes - External behavior identical
  • Refactoring/maintenance - Code improvements only
  • Tests still pass - All existing tests pass without modification

Lighter review: Internal changes may have lighter review requirements.

Instructions:

  1. Check off items as you complete them (change - [ ] to - [x])
  2. The PR checklist CI will verify these are completed

This checklist updates automatically when labels change, but preserves your checked boxes.

@OlivierHecart OlivierHecart added the internal Changes not included in the changelog label Jan 14, 2026
@codecov
Copy link

codecov bot commented Jan 14, 2026

Codecov Report

❌ Patch coverage is 83.57182% with 597 lines in your changes missing coverage. Please review.
✅ Project coverage is 72.64%. Comparing base (96a6e43) to head (7277cc0).
⚠️ Report is 35 commits behind head on regions.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
zenoh/src/net/routing/hat/broker/interests.rs 57.52% 110 Missing ⚠️
commons/zenoh-protocol/src/core/region.rs 50.00% 64 Missing ⚠️
zenoh/src/net/routing/dispatcher/tables.rs 32.22% 61 Missing ⚠️
zenoh/src/net/routing/hat/client/interests.rs 57.69% 55 Missing ⚠️
zenoh/src/net/routing/dispatcher/queries.rs 85.80% 46 Missing ⚠️
zenoh/src/net/routing/hat/broker/queries.rs 80.70% 44 Missing ⚠️
zenoh/src/net/routing/hat/client/token.rs 68.60% 27 Missing ⚠️
zenoh/src/net/routing/dispatcher/token.rs 81.74% 23 Missing ⚠️
zenoh/src/net/routing/hat/client/mod.rs 75.90% 20 Missing ⚠️
zenoh/src/net/routing/dispatcher/interests.rs 89.78% 19 Missing ⚠️
... and 26 more
Additional details and impacted files
@@             Coverage Diff             @@
##           regions    #2365      +/-   ##
===========================================
+ Coverage    71.96%   72.64%   +0.67%     
===========================================
  Files          390      395       +5     
  Lines        62769    58346    -4423     
===========================================
- Hits         45174    42384    -2790     
+ Misses       17595    15962    -1633     

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

@OlivierHecart OlivierHecart force-pushed the regions-tests branch 4 times, most recently from be36842 to 848c709 Compare January 16, 2026 14:45
@fuzzypixelz fuzzypixelz force-pushed the regions branch 8 times, most recently from d560731 to 86741af Compare January 26, 2026 15:01
@OlivierHecart OlivierHecart force-pushed the regions-tests branch 3 times, most recently from d82d75b to 4f03b4e Compare January 26, 2026 15:54
@fuzzypixelz fuzzypixelz force-pushed the regions branch 2 times, most recently from bfecfa4 to 4cf993a Compare January 27, 2026 13:39
@fuzzypixelz fuzzypixelz force-pushed the regions-tests branch 2 times, most recently from 29bc3a0 to 8d68387 Compare January 27, 2026 13:55
fuzzypixelz and others added 8 commits February 10, 2026 17:43
- Fix region handing of multicast transports and multicast peers
- Fix bug in `RouterBuilder` where client subregions were assigned the wrong hat
- Fix bug in `HatBaseTrait::init` where the wrong WhatAmI was used
Feature set:
- broker hat
- pub/sub w/ interests
- peer hat
- client hat
- query/reply w/ interests
- router hat
- liveliness
- adminspace
- multiple (peer) gateways
- region names

Limitations:
- no regionalized multicast scouting
- no router subregions of non-router regions
Feature set:
- broker hat
- pub/sub w/ interests
- peer hat
- client hat
- query/reply w/ interests
- router hat
- liveliness
- adminspace
- multiple (peer) gateways
- region names

Limitations:
- no regionalized multicast scouting
- no router subregions of non-router regions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal Changes not included in the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants