Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

[Bug] z_channels_test unit test breaks when built without multithreading #874

Closed
brandonho667 opened this issue Jan 31, 2025 · 1 comment · Fixed by #882
Closed

[Bug] z_channels_test unit test breaks when built without multithreading #874

brandonho667 opened this issue Jan 31, 2025 · 1 comment · Fixed by #882
Assignees
@sashacmc
Labels
bug Something isn't working

Comments

@brandonho667
Copy link

brandonho667 commented Jan 31, 2025 •
edited
Loading

Describe the bug

when running z_channels_test with multithreading disabled, there's an assertion error

z_channels_test: /home/zenoh-pico/tests/z_channels_test.c:153: sample_ring_channel_test_in_size: Assertion `strcmp(buf, "closed") == 0' failed.

at line 153. The only difference in execution I notice is here, not sure if that helps.

To reproduce

Disable multithreading in CMakeLists.txt, build and run z_channels_test

System info

  • Platform: Ubuntu 24.04
  • CPU: Intel(R) Xeon(R)
  • Zenoh version: bbb343c
@brandonho667 brandonho667 added the bug Something isn't working label Jan 31, 2025
@sashacmc
Copy link
Member

@brandonho667, thanks for the report, we'll look into it.

@sashacmc sashacmc self-assigned this Jan 31, 2025
@sashacmc sashacmc mentioned this issue Feb 3, 2025
Merged
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@sashacmc sashacmc

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix single thread ring channel ZettaScaleLabs/zenoh-pico
2 participants
@sashacmc @brandonho667