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

calibre: make entrypoint more generic #614

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

stavros-k
Copy link
Contributor

I'm not sure why it was /storage/elfstorage but volume was /media.
Let me know if you want to switch that back or update the volume.

Added a check that if a metadata.db is missing (eg if you mount a bind volume on top of the already created directory in the dockerfile)
Added variables for port and trusted IPs.

@stavros-k
Copy link
Contributor Author

With this as an entrypoint, I got it to work fine.

if [ ! -f /books/metadata.db ]; then
  echo 'Creating an empty calibre metadata database at [/books/metadata.db]'
  touch /books/metadata.db
fi

/usr/bin/calibre-server \
  --port=8085 \
  --enable-local-write \
  --disable-use-bonjour \
  --trusted-ips=192.168.0.0/16,172.16.0.0/12,10.0.0.0/8 \
  /books

I do see this

No write access to /home/elfie/.config/calibre using a temporary dir instead

But does not seem to affect usage.

If I set HOME=/tmp/home, the error goes away, but I don't see anything that needs to be persisted anyway.
I'm not sure if home is used for something else in the image and I don't want to "mess it up"

Copy link

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in two weeks if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale label Dec 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant