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

Add warning/option when installing Crowsnest w/ mjpegstreamer in tandem #374

Open
wolfnacht44 opened this issue Aug 22, 2023 · 0 comments
Open
Labels
improvement improvement of existing features/code
Milestone

Comments

@wolfnacht44
Copy link

Is your feature request related to a problem? Please describe

I didn't notice it, and realized ignorance with my issue at hand, during my migration to crowsnest from mjpeg-streamer
initially mjpeg-streamer and crowsnest initially conflict upon installation
enabling debug showed an issue with bind address, uninstalling mjpeg-streamer, or changing crowsnest port solved the issue

crowsnest: ERROR: Start of camera-streamer [cam rpi] failed! added for google crawlers

Describe the solution you'd like

  1. detection of mjpeg-streamer installation via kiauh.sh
  2. prompt "are you migrating from/to crowsnest/mjpeg-streamer"?
  3. prompt to auto uninstall crowsnest/mjpeg-streamer IF user responds [Y]es
  4. prompt WARNING stating conflict between the services if user enters [N]o - similar to installing fluidd/mainsail in tandem

Describe alternatives you've considered

Simply offering a Warning in place of the conflict described above
Ability to prompt for port change in config when installing either service if conflict is detected( IF user enters No at point 2 or 3- I believe this would be more complex)

Additional information

mjpeg-streamer installed via kiuah.sh

had to enable debug level logging to find the root cause
not realizing that Crowsnest and MJPEG-streamer were trying utilize the same port(8080)

Less knowledgeable users would be stuck on this particular issue when migrating or "testing" the different services in tandem, utilizing the script for its ease of use.

this is a thought I'd bring up incase there IS an issue with it in the future

verbose logging level tail in crowsnest

[08/22/23 11:39:16] crowsnest: INFO: Configuration of Section [cam rpi] looks good. Continue ...
[08/22/23 11:39:18] crowsnest: Starting camera-streamer with Device /base/soc/i2c0mux/i2c@1/imx477@1a ...
[08/22/23 11:39:18] crowsnest: ERROR: Start of camera-streamer [cam rpi] failed!

debug loging level tail in crowsnest

[08/22/23` 11:40:31] crowsnest: DEBUG: camera-streamer [cam rpi]: Parameters: --http-port=8080 --camera-path=/base/soc/i2c0mux/i2c@1/imx477@1a --camera-type=libcamera --camera-format=YUYV --camera-snapshot.height=1080 --camera-width=640 --camera-height=400 --camera-fps=15 --rtsp-port=8554 --camera-auto_reconnect=1
[08/22/23 11:40:32] crowsnest: DEBUG: camera-streamer [cam rpi]: bind: Address already in use
[08/22/23 11:40:32] crowsnest: DEBUG: camera-streamer [cam rpi]: /home/pi/crowsnest/bin/camera-streamer/camera-streamer Version: (ef8f6a4)
[08/22/23 11:40:32] crowsnest: DEBUG: camera-streamer [cam rpi]: xargs: /home/pi/crowsnest/bin/camera-streamer/camera-streamer: exited with status 255; aborting

@wolfnacht44 wolfnacht44 added the ✨ feature request New feature or request label Aug 22, 2023
@dw-0 dw-0 added improvement improvement of existing features/code and removed ✨ feature request New feature or request labels Aug 30, 2023
@dw-0 dw-0 added this to the next milestone Oct 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement improvement of existing features/code
Projects
None yet
Development

No branches or pull requests

2 participants