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 SiliconDust's HdHomerun standalone DVR engine Rockon #280

Closed
wants to merge 49 commits into from

Conversation

Hooverdan96
Copy link
Member

Fixes #278 .

General information on project
This pull request proposes to add a new rock-on for the following project:

name: HDHomeRun DVR engine from SiliconDust
website: https://www.silicondust.com
description: standalone DVR engine that work in conjunction with SiliconDust's HDHomeRun series of tuners that don't have the DVR built-in. Requires a valid subscription and recent HDHomeRun hardware
Information on docker image
docker image: https://hub.docker.com/repository/docker/jackdock96/hdh_dvr
is an official docker image available for this project?: Not at this time. I created this docker image to self-update with the latest engine and also offer multi-arch images to support Rockstor's direction of installing on ARM architecture systems, so the docker container doesn't have to be maintained in the repository too frequently.
Note: because the dvr engine uses multi-cast the Rockon is currently configured to use the net=host option. Unless one implements a macvlan or uses tools like pipework, multi-cast from LAN to container and vice-versa is not possible. I assume, Rockstor's Rocknets might be able to achieve that, but until we are completely cut over to 4.x on OpenSuse, this is the only option.
Plan is to switch to an official image if/when it becomes available

Checklist

  • Passes JSONlint validation
  • Entry added to root.json in alphabetical order (for new rock-on only)
  • "description" object lists and links to the docker image used
  • "description" object provides information on the image's particularities (advantage over another existing rock-on for the same project, for instance)
  • "website" object links to project's main website

@Hooverdan96
Copy link
Member Author

The amd64 version works very well on my environments. I have not been able to test the ARM architecture container versions on "real" hardware at this time, though.

Hooverdan96 and others added 19 commits March 17, 2021 10:41
switch to latest stable branch of docker container
Add new Rockon for latest stable LMS (Squeezebox Server) version.
Also added item for preferring multi-arch images for submission
As IPv6 is currently disabled, we also have to disable it for this rock-on via environment variables otherwise nginx will not start:

```
2021/02/11 20:35:31 [emerg] 14860#14860: socket() [::]:8080 failed (97: Address family not supported by protocol)
2021/02/11 20:35:32 [emerg] 14869#14869: socket() [::]:8080 failed (97: Address family not supported by protocol)
2021/02/11 20:35:33 [emerg] 14878#14878: socket() [::]:8080 failed (97: Address family not supported by protocol)
2021/02/11 20:35:34 [emerg] 14887#14887: socket() [::]:8080 failed (97: Address family not supported by protocol)
2021/02/11 20:35:35 [emerg] 14896#14896: socket() [::]:8080 failed (97: Address family not supported by protocol)
2021/02/11 20:35:36 [emerg] 14905#14905: socket() [::]:8080 failed (97: Address family not supported by protocol)
2021/02/11 20:35:37 [emerg] 14914#14914: socket() [::]:8080 failed (97: Address family not supported by protocol)
```
Fix nginx-proxy-manager for rockstor 4
Co-authored-by: FroggyFlox <30297881+FroggyFlox@users.noreply.github.com>
Co-authored-by: FroggyFlox <30297881+FroggyFlox@users.noreply.github.com>
Co-authored-by: FroggyFlox <30297881+FroggyFlox@users.noreply.github.com>
Co-authored-by: FroggyFlox <30297881+FroggyFlox@users.noreply.github.com>
Co-authored-by: FroggyFlox <30297881+FroggyFlox@users.noreply.github.com>
Following Wikipedia style guide
Add criteria for new vs. updated Rock-on
root.json Outdated Show resolved Hide resolved
@phillxnet
Copy link
Member

@Hooverdan96 Thanks for your continued effort in this repo by the way. Much appreciated.

Co-authored-by: Philip Guyton <phillxnet@users.noreply.github.com>
@Hooverdan96
Copy link
Member Author

Thanks @phillxnet, accepted your changes ... always good to have more eyes on this :).

freaktechnik and others added 15 commits October 1, 2021 10:06
Update node red volume according to container docs
As suggested in PR review.

Co-authored-by: FroggyFlox <30297881+FroggyFlox@users.noreply.github.com>
Titlecase

Co-authored-by: FroggyFlox <30297881+FroggyFlox@users.noreply.github.com>
Titlecase

Co-authored-by: FroggyFlox <30297881+FroggyFlox@users.noreply.github.com>
Co-authored-by: FroggyFlox <30297881+FroggyFlox@users.noreply.github.com>
Note that 9000 port needs to be used until net=host can be deprecated
Logitech Media Server (Squeezebox), move to latest release tag
The existing image used discourse/discourse is now superseded, and
presumably deprecated by: discourse/base, which needs quite a bit of
'special' treatment. And given we have had almost no reports of our
existing failure due to a non working image lets just remove the Rock-on
entirely.
…scourse_rock-on

remove existing discourse rock-on rockstor#185
Thanks to Hooverdan96 @thedrjones and @FroggyFlox for doing much of the
leg-work on this one via organisation and replacement rock-ons.

Removes definitions and root.json references for:
- NZBHydra we have NZBHydra2 now via pr rockstor#272
Fixes rockstor#273
- Zoneminder 1.29 & ZoneMinder-1.30 we have `latest` via rockstor#189 @magicalyak
Fixes rockstor#260
- TFTP Server redundant as we now have linuxserver/netbootxyz via rockstor#234
Fixes rockstor#257
- Plexpy image deprecated / replaced by Tautulli via rockstor#256
Fixes rockstor#255
- Koel image removed.
Fixes rockstor#254
- Cardigann replaced by our existing Jackett via rockstor#68 (thanks @porkie02)
Fixes rockstor#253
- Logitech(Squeezebox) replaced by LMS - Latest via rockstor#283
Fixes rockstor#281
@phillxnet
Copy link
Member

Just checked and this image was updated 24 days ago:
https://hub.docker.com/search?q=jackdock96%2Fhdh_dvr&type=image

But we have another host network requirement. But it's not quite a wild as say deluge that I recently closed an issue for to extent it to host networking:
#205

@FroggyFlox Your call on this one. I'm game if you are.

@Hooverdan96
Copy link
Member Author

@phillxnet, it was updated since I did the updating. I am maintaining a docker image (updating itself with the latest version of the executable whenever restarted). Fortunately, the docker container updates currently only involve building it with a new version of alpine. If/when an official docker is released (not sure if that ever happens), we can obviously go with that one.
and, yes, the net=host is again required for the upnp stuff, otherwise it cannot find the tuner. I've tried. I hope, there will be some type of "multicast" solution that easily integrates with docker in the future...

@Hooverdan96
Copy link
Member Author

I completely messed up my fork and ended up pushing not the right stuff. I am going to close this and replace it with #301

My apologies.

@Hooverdan96 Hooverdan96 closed this Dec 2, 2021
@Hooverdan96 Hooverdan96 deleted the HDHomeRunDVR_Rockon branch December 2, 2021 18:28
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.

Add HDHomeRun DVR engine Rockon
6 participants