Skip to content

Enable Zenoh router to make the post-install example work #5858

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 6 commits into from
Sep 18, 2025
Merged

Enable Zenoh router to make the post-install example work #5858

merged 6 commits into from
Sep 18, 2025

Conversation

@luisarizmendi
Copy link
Contributor

@luisarizmendi luisarizmendi commented Sep 16, 2025

Description

If you run the example as it appears in the DOC you won't get it working (listener do not receive anything).

Just by starting the Zenoh router it works:

ros2 run rmw_zenoh_cpp rmw_zenohd

I only updated the RHEL documentation since it is the system where I faced and solved the issue, but probably it will also happen in Ubuntu.

Did you use Generative AI?

NO

Additional Information

If you run without the router enabled you will get these warnings in both talker and listener:

[WARN] [1758031191.632663436] [rmw_zenoh_cpp]: Unable to connect to a Zenoh router. Have you started a router with `ros2 run rmw_zenoh_cpp rmw_zenohd`?
[WARN] [1758031191.633501357] [rmw_zenoh_cpp]: Unable to connect to a Zenoh router after 1 attempt(s). Please ensure that a Zenoh router is running and can be reached. You may increase the number of attempts to check for a router by setting the ZENOH_ROUTER_CHECK_ATTEMPTS environment variable. Proceeding with initialization but other peers will not discover or receive data from peers in this session until a router is started.

@ahcorde ahcorde added the backport-all backport at reviewers discretion; from rolling to all versions label Sep 16, 2025
ahcorde
ahcorde requested changes Sep 16, 2025
View reviewed changes
Copy link
Contributor

@ahcorde ahcorde left a comment

Choose a reason for hiding this comment

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

/home/runner/work/ros2_documentation/ros2_documentation/source/Installation/RHEL-Install-RPMs.rst:128:'any' reference target not found: Zenoh

@luisarizmendi @ahcorde
Update source/Installation/RHEL-Install-RPMs.rst
3af1086 
Co-authored-by: Alejandro Hernández Cordero <[email protected]>
Signed-off-by: Luis Javier Arizmendi Alonso <[email protected]>
fujitatomoya
fujitatomoya reviewed Sep 17, 2025
View reviewed changes
luisarizmendi and others added 2 commits September 17, 2025 16:57
@luisarizmendi @fujitatomoya
Update source/Installation/RHEL-Install-RPMs.rst
00fd6b4 
Co-authored-by: Tomoya Fujita <[email protected]>
Signed-off-by: Luis Javier Arizmendi Alonso <[email protected]>
@luisarizmendi @fujitatomoya
Update source/Installation/RHEL-Install-RPMs.rst
8f34ae1 
Co-authored-by: Tomoya Fujita <[email protected]>
Signed-off-by: Luis Javier Arizmendi Alonso <[email protected]>
@fujitatomoya
Copy link
Collaborator

fujitatomoya commented Sep 17, 2025

@ahcorde i will leave this to you, can you approve and merge?

@fujitatomoya fujitatomoya mentioned this pull request Sep 17, 2025
Merged
@ahcorde ahcorde merged commit b684b72 into ros2:rolling Sep 18, 2025
5 checks passed
mergify bot pushed a commit that referenced this pull request Sep 18, 2025
@luisarizmendi @ahcorde
@fujitatomoya @mergify
Enable Zenoh router to make the post-install example work (#5858)
7da73af 
Signed-off-by: Luis Javier Arizmendi Alonso <[email protected]>
Co-authored-by: Alejandro Hernández Cordero <[email protected]>
Co-authored-by: Tomoya Fujita <[email protected]>
(cherry picked from commit b684b72)
mergify bot pushed a commit that referenced this pull request Sep 18, 2025
@luisarizmendi @ahcorde
@fujitatomoya @mergify
Enable Zenoh router to make the post-install example work (#5858)
6f2d84e 
Signed-off-by: Luis Javier Arizmendi Alonso <[email protected]>
Co-authored-by: Alejandro Hernández Cordero <[email protected]>
Co-authored-by: Tomoya Fujita <[email protected]>
(cherry picked from commit b684b72)
This was referenced Sep 18, 2025
Merged
Merged
mergify bot pushed a commit that referenced this pull request Sep 18, 2025
@luisarizmendi @ahcorde
@fujitatomoya @mergify
Enable Zenoh router to make the post-install example work (#5858)
3054c4a 
Signed-off-by: Luis Javier Arizmendi Alonso <[email protected]>
Co-authored-by: Alejandro Hernández Cordero <[email protected]>
Co-authored-by: Tomoya Fujita <[email protected]>
(cherry picked from commit b684b72)
@mergify mergify bot mentioned this pull request Sep 18, 2025
Merged
ahcorde added a commit that referenced this pull request Sep 18, 2025
@mergify @luisarizmendi
@ahcorde @fujitatomoya
Enable Zenoh router to make the post-install example work (#5858) (#5863
d6973fd 
)

(cherry picked from commit b684b72)

Signed-off-by: Luis Javier Arizmendi Alonso <[email protected]>
Co-authored-by: Luis Javier Arizmendi Alonso <[email protected]>
Co-authored-by: Alejandro Hernández Cordero <[email protected]>
Co-authored-by: Tomoya Fujita <[email protected]>
ahcorde added a commit that referenced this pull request Sep 18, 2025
@mergify @luisarizmendi
@ahcorde @fujitatomoya
Enable Zenoh router to make the post-install example work (#5858) (#5862
93e5063 
)

(cherry picked from commit b684b72)

Signed-off-by: Luis Javier Arizmendi Alonso <[email protected]>
Co-authored-by: Luis Javier Arizmendi Alonso <[email protected]>
Co-authored-by: Alejandro Hernández Cordero <[email protected]>
Co-authored-by: Tomoya Fujita <[email protected]>
ahcorde added a commit that referenced this pull request Sep 18, 2025
@mergify @luisarizmendi
@ahcorde @fujitatomoya
Enable Zenoh router to make the post-install example work (#5858) (#5861
03fd2ec 
)

(cherry picked from commit b684b72)

Signed-off-by: Luis Javier Arizmendi Alonso <[email protected]>
Co-authored-by: Luis Javier Arizmendi Alonso <[email protected]>
Co-authored-by: Alejandro Hernández Cordero <[email protected]>
Co-authored-by: Tomoya Fujita <[email protected]>
This was referenced Sep 18, 2025
Merged
Merged
@mergify mergify bot mentioned this pull request Sep 18, 2025
Merged
@luisarizmendi luisarizmendi deleted the doc-fix-example branch September 22, 2025 08:05
This was referenced Oct 17, 2025
Closed
Merged
MrBearing added a commit to MrBearing/ros2_documentation that referenced this pull request Oct 21, 2025
@MrBearing
Revert "Enable Zenoh router to make the post-install example work (ro…
9b15172 
…s2#5858)"

This reverts commit b684b72.
fujitatomoya pushed a commit to MrBearing/ros2_documentation that referenced this pull request Oct 21, 2025
@MrBearing @fujitatomoya
Revert "Enable Zenoh router to make the post-install example work (ro…
15af3fb 
…s2#5858)"

This reverts commit b684b72.
fujitatomoya pushed a commit that referenced this pull request Oct 21, 2025
@MrBearing
Fixing misleading Zenoh usage in try-some-example #5954 (#5957)
ecc26c2 
* Revert "Enable Zenoh router to make the post-install example work (#5858)"

This reverts commit b684b72.

* Revert "Enable zenohd to make the post-install example work for Ubuntu Debian. (#5860)"

This reverts commit 86d8c1a.

* Added link to RMW implementations page.
mergify bot pushed a commit that referenced this pull request Oct 21, 2025
@MrBearing @mergify
Fixing misleading Zenoh usage in try-some-example #5954 (#5957)
8a05530 
* Revert "Enable Zenoh router to make the post-install example work (#5858)"

This reverts commit b684b72.

* Revert "Enable zenohd to make the post-install example work for Ubuntu Debian. (#5860)"

This reverts commit 86d8c1a.

* Added link to RMW implementations page.

(cherry picked from commit ecc26c2)
mergify bot pushed a commit that referenced this pull request Oct 21, 2025
@MrBearing @mergify
Fixing misleading Zenoh usage in try-some-example #5954 (#5957)
913da24 
* Revert "Enable Zenoh router to make the post-install example work (#5858)"

This reverts commit b684b72.

* Revert "Enable zenohd to make the post-install example work for Ubuntu Debian. (#5860)"

This reverts commit 86d8c1a.

* Added link to RMW implementations page.

(cherry picked from commit ecc26c2)
mergify bot pushed a commit that referenced this pull request Oct 21, 2025
@MrBearing @mergify
Fixing misleading Zenoh usage in try-some-example #5954 (#5957)
9eaf035 
* Revert "Enable Zenoh router to make the post-install example work (#5858)"

This reverts commit b684b72.

* Revert "Enable zenohd to make the post-install example work for Ubuntu Debian. (#5860)"

This reverts commit 86d8c1a.

* Added link to RMW implementations page.

(cherry picked from commit ecc26c2)

# Conflicts:
#	source/Installation/RHEL-Install-RPMs.rst
#	source/Installation/Ubuntu-Install-Debs.rst
fujitatomoya pushed a commit that referenced this pull request Oct 21, 2025
@mergify @MrBearing
Fixing misleading Zenoh usage in try-some-example #5954 (#5957) (#5976)
80f29f0 
* Revert "Enable Zenoh router to make the post-install example work (#5858)"

This reverts commit b684b72.

* Revert "Enable zenohd to make the post-install example work for Ubuntu Debian. (#5860)"

This reverts commit 86d8c1a.

* Added link to RMW implementations page.

(cherry picked from commit ecc26c2)

Co-authored-by: Takumi Okamoto <[email protected]>
fujitatomoya pushed a commit that referenced this pull request Oct 21, 2025
@mergify @MrBearing
Fixing misleading Zenoh usage in try-some-example #5954 (#5957) (#5975)
d617bb3 
* Revert "Enable Zenoh router to make the post-install example work (#5858)"

This reverts commit b684b72.

* Revert "Enable zenohd to make the post-install example work for Ubuntu Debian. (#5860)"

This reverts commit 86d8c1a.

* Added link to RMW implementations page.

(cherry picked from commit ecc26c2)

Co-authored-by: Takumi Okamoto <[email protected]>
fujitatomoya added a commit that referenced this pull request Oct 21, 2025
@mergify @MrBearing @fujitatomoya
Fixing misleading Zenoh usage in try-some-example #5954 (backport #5957…
e7daea1 
…) (#5977)

* Fixing misleading Zenoh usage in try-some-example #5954 (#5957)

* Revert "Enable Zenoh router to make the post-install example work (#5858)"

This reverts commit b684b72.

* Revert "Enable zenohd to make the post-install example work for Ubuntu Debian. (#5860)"

This reverts commit 86d8c1a.

* Added link to RMW implementations page.

(cherry picked from commit ecc26c2)

# Conflicts:
#	source/Installation/RHEL-Install-RPMs.rst
#	source/Installation/Ubuntu-Install-Debs.rst

* resolve conflicts.

Signed-off-by: Tomoya.Fujita <[email protected]>

---------

Signed-off-by: Tomoya.Fujita <[email protected]>
Co-authored-by: Takumi Okamoto <[email protected]>
Co-authored-by: Tomoya.Fujita <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fujitatomoya fujitatomoya fujitatomoya approved these changes

@ahcorde ahcorde ahcorde approved these changes

Assignees

@luisarizmendi luisarizmendi

Labels

backport-all backport at reviewers discretion; from rolling to all versions

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@luisarizmendi @fujitatomoya @ahcorde