Skip to content

core: start-blueos-core: Remove zenoh from mavlink-camera-manager#3929

Merged
joaoantoniocardoso merged 1 commit into
bluerobotics:masterfrom
patrickelectric:remove-zenoh-mcm
May 11, 2026
Merged

core: start-blueos-core: Remove zenoh from mavlink-camera-manager#3929
joaoantoniocardoso merged 1 commit into
bluerobotics:masterfrom
patrickelectric:remove-zenoh-mcm

Conversation

@patrickelectric
Copy link
Copy Markdown
Member

@patrickelectric patrickelectric commented May 11, 2026

This is a temporary solution until will work to a better video recording/extraction

Summary by Sourcery

Enhancements:

  • Simplify mavlink-camera-manager startup by dropping zenoh-related usage in the core startup sequence.

This is a temporary solution until will work to a better video recording/extraction

Signed-off-by: Patrick José Pereira <patrickelectric@gmail.com>
Copy link
Copy Markdown

@sourcery-ai sourcery-ai Bot left a comment

Choose a reason for hiding this comment

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

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@github-actions
Copy link
Copy Markdown

Automated PR Review

0. Summary

  • Verdict: LGTM :shipit:

Drops the --zenoh flag from the mavlink-camera-manager invocation in core/start-blueos-core:120. The zenohd service entry on line 128 is left intact so other zenoh consumers (e.g. commonwealth/utils/logs.py) are unaffected. PR body flags this as a temporary measure pending a better video recording/extraction path.

No further comments, nice job 👍

Generated by PR Review Bot. This is advisory, a human reviewer must still approve.

@joaoantoniocardoso joaoantoniocardoso merged commit 6c94071 into bluerobotics:master May 11, 2026
8 checks passed
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.

2 participants