Fix shm size calculation #5123
Merged
Fix shm size calculation #5123
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
✅ Deploy Preview for frigate-docs ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
|
Oh wait. I just realized that this math is for a single camera! |
NickM-27
approved these changes
Jan 17, 2023
|
@NickM-27 I re-wrote this PR. If you could take a look again, please. |
NickM-27
approved these changes
Jan 17, 2023
|
Oh. I just had an idea. I'll put up some instructions on how to change the default shm-size when using HassOS. Sorry about that. |
|
Actually never mind. This is too complicated. I didn't change anything. |
|
Ok, too bad. This can't be changed for add-ons at all! Since HassOS 9.0 they made the |
|
Actually, good news! They give you access to the whole memory or 50% of it, or a big amount anyway, it seems. ❯ docker exec -ti addon_ccab4aaf_frigate df -h /dev/shm
Filesystem Size Used Avail Use% Mounted on
tmpfs 3.9G 9.3M 3.8G 1% /dev/shm |
|
I added a note about it, we don't need users to think that they need to tinker with it. |
blakeblackshear
approved these changes
Jan 18, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I think they were not taking the log files into account (10MB for each the three).