Added filebrowser #18

Merged

sudo-kraken merged 2 commits from features/filebrowser into main

2024-12-05 20:06:30 +00:00

sudo-kraken commented

2024-12-04 21:31:42 +00:00

(Migrated from github.com)

No description provided.

redbeardymcgee (Migrated from github.com) reviewed 2024-12-05 03:15:33 +00:00

redbeardymcgee (Migrated from github.com) left a comment

This is a new one for me, thanks for showing it to me!

quadlets/filebrowser/filebrowser.container

redbeardymcgee (Migrated from github.com) commented

2024-12-05 03:11:48 +00:00

I want to clarify what this is about and see if there's something we can do better about that.

There's another quadlet here that had a quirk similar to this. I need to check on which that was and revisit it also. One that stands out is calibre-web because it wants a metadata.db from an existing calibre library. I added a README.md for that situation, but I seem to remember another one that needed to be bootstrapped.

I want to clarify what this is about and see if there's something we can do better about that. There's another quadlet here that had a quirk similar to this. I need to check on which that was and revisit it also. One that stands out is `calibre-web` because it wants a `metadata.db` from an existing `calibre` library. I added a `README.md` for that situation, but I seem to remember another one that needed to be bootstrapped.

redbeardymcgee (Migrated from github.com) commented

2024-12-05 03:14:30 +00:00

These are holdovers from an outmoded method of handling docker user/group stuff inside containers. podman should not require this anymore, and 1000 cannot be hardcoded. Even though upstreams and lsio containers use it as an example all the time, we have subuids and subgids, and 1000 is only correct if you are the only user on the system.

These are holdovers from an outmoded method of handling docker user/group stuff inside containers. `podman` should not require this anymore, and `1000` cannot be hardcoded. Even though upstreams and lsio containers use it as an example all the time, we have subuids and subgids, and `1000` is only correct if you are the only user on the system.

sudo-kraken (Migrated from github.com) reviewed 2024-12-05 07:35:34 +00:00

quadlets/filebrowser/filebrowser.container

sudo-kraken (Migrated from github.com) commented

2024-12-05 07:35:34 +00:00

I did try and add a post exec with a symlink but that failed I can look into this again though and try work out a solution.

sudo-kraken commented

2024-12-05 12:34:08 +00:00

(Migrated from github.com)

Fixed the volumes issue please review.

redbeardymcgee commented

2024-12-05 13:53:54 +00:00

(Migrated from github.com)

Looks like you found a different provider for the container that has more sane behavior regarding the mounts, and now we can skip that bootstrap step?

sudo-kraken commented

2024-12-05 14:33:00 +00:00

(Migrated from github.com)

Looks like you found a different provider for the container that has more sane behavior regarding the mounts, and now we can skip that bootstrap step?

correct, this is a very popular and maintained image mirror with more sane mounting approaches.

> Looks like you found a different provider for the container that has more sane behavior regarding the mounts, and now we can skip that bootstrap step? correct, this is a very popular and maintained image mirror with more sane mounting approaches.