Skip to content

Frigate#394

Draft
mcmikemn wants to merge 7 commits into
masterfrom
frigate
Draft

Frigate#394
mcmikemn wants to merge 7 commits into
masterfrom
frigate

Conversation

@mcmikemn

Copy link
Copy Markdown
Collaborator

Implements #393

@mcmikemn mcmikemn added the enhancement New feature or request label Feb 14, 2025
@mcmikemn mcmikemn self-assigned this Feb 14, 2025
@mcmikemn mcmikemn linked an issue Feb 14, 2025 that may be closed by this pull request
@mcmikemn

Copy link
Copy Markdown
Collaborator Author

@EnigmaCurry: this is working in that Frigate gets installed, the basics get configured, and you can log into it. I don't have any cameras to test video streaming so it's possible that I'll need to add something to docker-compose.instance.yaml for RTSP and WebRTC (https://docs.frigate.video/frigate/installation/#docker).

Comment thread frigate/Makefile Outdated
@echo

.PHONY: disable-tls # Disable TLS on Frigate's webserver
disable-tls:

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Normally in d.ry we setup a "config" container to setup any non env var enabled config. This would set the config file before the service starts. Can we do that instead?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

That is, of course, better way. Fixed in 10c27c6

@EnigmaCurry

Copy link
Copy Markdown
Owner

For the extra ports needed, theres a few options:

Add entrypoint through traefik. This has the benefit that you can bind to specific host ip address and apply an ip whitelist. But this has the overhead of passing large video frames through traefik.

Add the port mapping directly in docker compose. We dont normally do this, but it might be preferred for constant video ingest.(no traefik overhead) however you lose the ip filter so you would want to be sure to bind it to a specific host ip address so its not publicly accessible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Frigate.

2 participants