-
-
Notifications
You must be signed in to change notification settings - Fork 234
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
✨ [REQUEST] Jellyfish NAS: omit /config/addons_config/jellyfin/data/metadata
from backup
#1048
Comments
Why don't you just avoid the jellyfin addon from the backup? Do you need to keep safe all this metadata? |
I like to backup the jellyfin configuration, but definitely not this metadata. Ideally, it would be great if I could just exclude |
As is said in the thread you’ve linked :
alas it is still the same ; the true solution would be to be to request to the HA team to implement something such as a .exclude feature. because if I store the metadata in /data, it will still take place in backups, and if I store it in a non rémanent folder people will be unhappy as it would rebuild metadata at each start… |
|
Yes but I can’t implement that it must be done in the ha supervisor |
Ok I have a solution, I'll check |
Hi, I've pushed a test version could you please check ? I've moved those folders to /data, which allows to take advantage from the embedded feature from HA to exclude folders stored in /data |
Typo, I'll push a test2 |
Oh! Good catch! Great idea! |
What do I need to do? I moved |
Oh, everything should have been done automatically ! And your current metadata folder should have moved to /data/data/metadata (not pretty but easiest for me to keep the existing structure) |
How about the backup? Should I be excluding certain folders etc? |
Nope, I'm still getting 1.5GB backup |
Wait, correction. It works! Something else got bloated, let me fix on my end. Thank you! |
Well this would be most unlikely… one add on cannot communicate with others data folders. I would recommend that you put back this folder in the initial jellyfin/data/metadata, then restart the jellyfin addon and check that the folder disappears. Thanks ! |
Hi, ingress is not working with jellyfin in the latest version. Was it previously ? Thanks |
what do you mean by |
Accessing jellyfin from the sidebar in HA, instead of using ip:port |
If you use HTTP, ingress works, but if configured for HTTPS, ingress stops working. |
Thanks |
I am thinking maybe it is good to use the same strategy in the emby addon! :) |
Using caddy also cuts down on the need to to deal with the quirks of each addon individually, e.g. Navidrome needs the cert in PFX format... 🤦♂ |
Seems last changes with folders fully broken metadata in my case. Additionally, excluding metadata doesn't seem like a good solution. Sometimes this metadata is unique, such as custom pictures. |
Then I'll put back things how they were and think about it again ;-) |
A way to test without modying addon is to use my custom script mod : https://github.com/alexbelgium/hassio-addons/wiki/Add%E2%80%90ons-feature-:-customisation To do that, just create the file : /config/addons_autoscript/jellyfin.sh and put inside something like (not tested) : #!/bin/bash
echo "Starting script"
# Create folder if not existing
mkdir -p /data/metadata
# Add permissions to LSIO default user
chown abc:abc /data/metadata
chmod 755 /data/metadata
# Remove folder if existing
if [ -d /config/addons_config/jellyfin/metadata ]; then rm -r /config/addons_config/jellyfin/metadata; fi
ln -s /data/metadata /config/addons_config/jellyfin
echo "Done" |
|
|
Indeed... I've put the code as it was, and then propose to deal with avoiding the backup with my specific script above Btw, HomeAssistant has once again change the logic of addons and the /config won't be available anymore so it will likely break lots of things... |
@alexbelgium what's changed? any links? Thanks. |
Sorry it was this push fbc68bd normally the latest version should restore the old metadata behavior |
@alexbelgium I didn't update to the latest version jet. Just downgrade to .11 version and restore metadata from backup. Personally, it was not a problem for me. Thank you for your hard work and the time you spend! |
@alexbelgium May I suggest having a JF lite? The difference is only in the config file, correct? If it is too much trouble, don't worry. I will create my own fork. Thank you. |
You mean just for this metadata thing? The easiest would be to apply what I described in this post : #1048 (comment) it is the best way to customize an existing addon apart from forking it |
Pardon me, too many things happening on my side to pay close attention.
|
Hi, summary is :
|
I've followed the instructions above to move the metadata (and cache /config/addons_config/jellyfin/cache) which took it out of the core backup. The data is now included as part of the jellyfin addon, which made it easy to exclude from backups. Is that how the workaround above is meant to work? Just want to be sure i've it set up correctly and that theres no way of backing up jellyfin too (but not the data).
Thanks |
Checked
Which addon?
Jellyfish NAS
Is your feature request related to a problem? Please describe
Yes, the
/config/addons_config/jellyfin/data/metadata
folder is over 2GB and that bulks up my backup which was originally less than 500MBDescribe the solution you'd like
I like to have the backup as small as possible
The text was updated successfully, but these errors were encountered: