Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Config: tune some values for ROS 2 use cases #509

Merged
merged 2 commits into from
Mar 7, 2025

Conversation

Mallets
Copy link
Contributor

@Mallets Mallets commented Mar 6, 2025

ROS 2 systems can experience a large number of nodes (e.g. >200) starting all together.
This may lead to having a large number of connections be initialised at the same time during the startup phase.

This PR updates some Zenoh default limits that are deemed too small for the ROS 2 cases for both the Router and the Nodes. The reason for each update is documented directly in the config files.

No behaviour change is expected to be introduced by this PR, just a smoother out-of-the-box experience for large use cases.

Copy link
Member

@Yadunund Yadunund left a comment

Choose a reason for hiding this comment

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

LGTM with usual CI passing.

@Yadunund Yadunund merged commit bab4c93 into ros2:rolling Mar 7, 2025
4 of 5 checks passed
@Yadunund
Copy link
Member

Yadunund commented Mar 7, 2025

@Mergifyio backport jazzy humble

Copy link

mergify bot commented Mar 7, 2025

backport jazzy humble

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Mar 7, 2025
…er of Nodes (>200) (#509)

Co-authored-by: Julien Enoch <[email protected]>
Co-authored-by: Yadunund <[email protected]>
(cherry picked from commit bab4c93)
mergify bot pushed a commit that referenced this pull request Mar 7, 2025
…er of Nodes (>200) (#509)

Co-authored-by: Julien Enoch <[email protected]>
Co-authored-by: Yadunund <[email protected]>
(cherry picked from commit bab4c93)
Yadunund pushed a commit that referenced this pull request Mar 7, 2025
…er of Nodes (>200) (#509) (#515)

Co-authored-by: Julien Enoch <[email protected]>
Co-authored-by: Yadunund <[email protected]>
(cherry picked from commit bab4c93)

Co-authored-by: Luca Cominardi <[email protected]>
ahcorde pushed a commit that referenced this pull request Mar 7, 2025
…er of Nodes (>200) (#509) (#516)

Co-authored-by: Julien Enoch <[email protected]>
Co-authored-by: Yadunund <[email protected]>
(cherry picked from commit bab4c93)

Co-authored-by: Luca Cominardi <[email protected]>
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.

3 participants