Skip to content

Comments

refactor: single-camera-per-process rtsp-camera node#3

Merged
edgarriba merged 2 commits intomainfrom
feat/single-camera-per-process
Feb 1, 2026
Merged

refactor: single-camera-per-process rtsp-camera node#3
edgarriba merged 2 commits intomainfrom
feat/single-camera-per-process

Conversation

@edgarriba
Copy link
Member

Summary

  • Refactor rtsp-camera from multi-camera-per-process to single-camera-per-process architecture
  • Flatten Config struct (remove Vec<CameraConfig> wrapper), add validation with bounds checking
  • Replace cameras_node binary with rtsp_camera_node following the system-telemetry CLI pattern (-c, -e)
  • Add scoped health topic per instance: bubbaloop/{scope}/{machine}/health/rtsp-camera-{name}
  • Add example per-camera configs (entrance.yaml, terrace.yaml)

Multi-Instance Support

Each camera runs as its own process with its own config file. Register multiple instances with the daemon:

bubbaloop node add ./rtsp-camera --name camera-entrance --config configs/entrance.yaml
bubbaloop node add ./rtsp-camera --name camera-terrace --config configs/terrace.yaml
bubbaloop node start camera-entrance
bubbaloop node start camera-terrace

Test plan

  • Clean cargo build --release succeeds
  • cargo test -- 9/9 pass (config parsing, validation, env override)
  • Manual: register instance with daemon, verify independent start/stop/logs

🤖 Generated with Claude Code

edgarriba and others added 2 commits February 1, 2026 16:08
Flatten multi-camera config to single-camera Config struct with
validation. Replace cameras_node binary with rtsp_camera_node that
follows the system-telemetry pattern (-c config.yaml, -e endpoint).
Each camera instance gets its own process, config file, and scoped
health topic (rtsp-camera-{name}).

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Remove needless Default::default() struct update and allow
too_many_arguments on private compressed_task function.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@edgarriba edgarriba merged commit 4fa07a7 into main Feb 1, 2026
6 checks passed
@edgarriba edgarriba deleted the feat/single-camera-per-process branch February 1, 2026 15:21
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.

1 participant