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

Zenoh fails to reconnect when running with lowlatency and without scouting/gossip #1024

Open
esemeniuc opened this issue May 13, 2024 · 3 comments · May be fixed by #1030
Open

Zenoh fails to reconnect when running with lowlatency and without scouting/gossip #1024

esemeniuc opened this issue May 13, 2024 · 3 comments · May be fixed by #1030
Labels
bug Something isn't working

Comments

@esemeniuc
Copy link

Describe the bug

I'm running pub/sub example, meant for servers running in production. Multicast and gossip expose more risk, so they are disabled. I'm have trouble reconnecting after the publisher restarts when in low_latency mode.

To reproduce

Please see this diff: https://github.com/eclipse-zenoh/zenoh/compare/0.11.0-rc.2...jito-labs:zenoh:eric/bug-repro?expand=1

  1. Start Publisher on machine 1
  2. Start Subscriber on machine 2, let it receive messages
  3. Restart Publisher on machine 1
  4. Subscriber never reconnects

System info

Arch, linux 6.6.28-1-lts
Rust version: using default in toolchain file

@esemeniuc esemeniuc added the bug Something isn't working label May 13, 2024
@YuanYuYuan
Copy link
Contributor

YuanYuYuan commented May 14, 2024

Hi @esemeniuc! Thanks for reporting the bug. I can confirm the issue doesn't occur with the default universal transport. I'll investigate the reason.

@esemeniuc
Copy link
Author

I noticed that the publisher can use low latency mode without issues. The subscriber is the one bugging out.

@YuanYuYuan
Copy link
Contributor

I noticed that the publisher can use low latency mode without issues. The subscriber is the one bugging out.

Yes, it is. We didn't handle the link closing properly. #1030 should fix this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants