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

feat: support build docker image #306

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

feat: support build docker image #306

wants to merge 2 commits into from

Conversation

pkking
Copy link

@pkking pkking commented Jun 16, 2024

This PR want to support building docker image for this awesome project, maybe more steps can be done like publish package with github action :)

PS: will be happy to make more contributions!

Copy link
Owner

@simulot simulot left a comment

Choose a reason for hiding this comment

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

This is cool. Thank you.
The initial idea of immich-go was to not bother tandom users with npm, docker, etc... :-)

But apparently there is an use case for docker...

Do you know how to publish the package with github actions?

@pkking
Copy link
Author

pkking commented Jun 17, 2024

This is cool. Thank you. The initial idea of immich-go was to not bother tandom users with npm, docker, etc... :-)

But apparently there is an use case for docker...

Do you know how to publish the package with github actions?

Will try to add a workflow file later, ref: https://github.com/pkking/copr_docker/blob/main/.github/workflows/keygen-signd-publish.yml#L48-L110

@pkking pkking force-pushed the main branch 2 times, most recently from 2dccfb3 to 26e3f9a Compare June 17, 2024 12:21
@pkking
Copy link
Author

pkking commented Jun 17, 2024

@simulot This pr contains a new workflow that can publish a docker image to docker hub && ghcr registry, you just need to add two github action secret DOCKERHUB_USERNAME and DOCKERHUB_TOKEN

Here's the newest image i pushed from the forked repo action

@simulot
Copy link
Owner

simulot commented Jun 17, 2024

I have more questions about this? Can we have discussion on Discord or by email?

@pkking
Copy link
Author

pkking commented Jun 18, 2024

I have more questions about this? Can we have discussion on Discord or by email?

@simulot Sure thing, discord id: lcr1221 and TZ: GMT+8, if not available, can email me using the address from the PR commit :)

@erkexzcx
Copy link
Contributor

Why would someone need Docker image for this?. 🤔

Also, why would you use anything other than scratch as base image? My example (that also needs ca-certificates for Golang): https://github.com/erkexzcx/valetudopng/blob/main/Dockerfile

@erkexzcx
Copy link
Contributor

@simulot you can also use ghcr.io (GitHub's registry) instead of Dockerhub.

      - name: Login to GitHub Container Registry
        uses: docker/login-action@v3
        with:
          registry: ghcr.io
          username: ${{ github.actor }}
          password: ${{ secrets.GITHUB_TOKEN }}

@simulot
Copy link
Owner

simulot commented Jun 21, 2024

@erkexzcx
Thanks, I prefer this option.

Is it possible to leverage github actions to generate binaries for all platforms? triggered by tagging the main branch?

@erkexzcx
Copy link
Contributor

@pkking what is the benefit of Docker to begin with? Here is why I believe Docker images are not needed:

  1. This app does not run as a service/daemon (unless I am missing something?)
  2. This app has zero dependencies, literally a single statically linked binary.
  3. Putting this binary into a container would end up with...the same binary, just with extra steps. 🤷

@pkking
Copy link
Author

pkking commented Jun 22, 2024

Why would someone need Docker image for this?. 🤔

Also, why would you use anything other than scratch as base image? My example (that also needs ca-certificates for Golang): https://github.com/erkexzcx/valetudopng/blob/main/Dockerfile

Hi @erkexzcx, my use case is i want to run immich-go in cronjob, each time the job scheduled, will need to upgrade immich and immich-go both firstly and i didnt find a one click upgrade script except using docker, so that's it :)

@pkking
Copy link
Author

pkking commented Jun 22, 2024

@simulot you can also use ghcr.io (GitHub's registry) instead of Dockerhub.

      - name: Login to GitHub Container Registry
        uses: docker/login-action@v3
        with:
          registry: ghcr.io
          username: ${{ github.actor }}
          password: ${{ secrets.GITHUB_TOKEN }}

https://github.com/simulot/immich-go/pull/306/files#diff-551d1fcf87f78cc3bc18a7b332a4dc5d8773a512062df881c5aba28a6f5c48d7R47-R53 this PR will push the image to both dockerhub and ghcr

@simulot
Copy link
Owner

simulot commented Jun 22, 2024

Hi @erkexzcx, my use case is i want to run immich-go in cronjob, each time the job scheduled, will need to upgrade immich and immich-go both firstly and i didnt find a one click upgrade script except using docker, so that's it :)

That's legit. But this will put the maintenance and the support for this release on me. I don't master the docker environnement enough for providing a good support.

What if you release the docker image on your own, synchronized with immich-go release? I'll add a section in the readme file that points to your repo?

@erkexzcx
Copy link
Contributor

my use case is i want to run immich-go in cronjob, each time the job scheduled

Instead of using Docker....

0 3 * * * docker run -it simulot/immich-go <args>

you can totally use binary just in a same way...

0 3 * * * /usr/local/bin/immich-go <args>

@simulot not that it would bring more maintenance, but there is barely any benefit of providing Docker image for a tool like this. Imagine this would have been nodejs project - totally makes sense to provide Docker image as user wouldn't need to install nodejs and with Docker you can ensure that user uses the right nodejs version, the right configuration, you provide isolated and consistent environment etc. With Golang, it's just a single binary, no external deps and personally I don't see any reason to provide Docker image.

Certain services, including the ones that I (barelly) maintain, they are running as services/daemons and even tho they can be used as regular binaries, I also provide Docker images because it's way more convenient to have all your services at your home server to be running under a single "roof" which is Docker service. But again - those are services/daemons that run 24/7, while this tool is supposed to be ran once and that's it.

@simulot
Copy link
Owner

simulot commented Jun 22, 2024

@erkexzcx your point is valid. I have started immich-go exactly because I wouldn't install node.js and all other dependencies on my laptop. A golang program is a perfect fit for that need.

Some users are using immich-go not for the google photos import but for synchronise regularly immich with another source of photos. They run immich-go as a cron task. Certainly a niche usage. Immich-go is open source, and every one can contribute. So why not. @pkking can maintain the image on is own.

@pkking
Copy link
Author

pkking commented Jun 22, 2024

my use case is i want to run immich-go in cronjob, each time the job scheduled

Instead of using Docker....

0 3 * * * docker run -it simulot/immich-go <args>

you can totally use binary just in a same way...

0 3 * * * /usr/local/bin/immich-go <args>

@simulot not that it would bring more maintenance, but there is barely any benefit of providing Docker image for a tool like this. Imagine this would have been nodejs project - totally makes sense to provide Docker image as user wouldn't need to install nodejs and with Docker you can ensure that user uses the right nodejs version, the right configuration, you provide isolated and consistent environment etc. With Golang, it's just a single binary, no external deps and personally I don't see any reason to provide Docker image.

Certain services, including the ones that I (barelly) maintain, they are running as services/daemons and even tho they can be used as regular binaries, I also provide Docker images because it's way more convenient to have all your services at your home server to be running under a single "roof" which is Docker service. But again - those are services/daemons that run 24/7, while this tool is supposed to be ran once and that's it.

@erkexzcx Thats right, so thats an XY problem, the actual reason i want to use docker is because i want to upgrade immich-go to latest version before each cronjob started, for immich-go in docker, just

* * * * * sync_photos.sh

then

cat sync_photos.sh
#!/bin/bash
docker rmi  simulot/immich-go
docker run -t --rm  simulot/immich-go upload <path to upload>

Without a docker image, i can't find a way to do this, if anyone provided a one Liner script to achive this, that would be The One as well :)

And yes, @simulot i'd like maintain the docker and github actions stuff is this PR get merged

@erkexzcx
Copy link
Contributor

@erkexzcx Thats right, so thats an XY problem, the actual reason i want to use docker is because i want to upgrade immich-go to latest version before each cronjob started, for immich-go in docker, just

* * * * * sync_photos.sh

then

cat sync_photos.sh
#!/bin/bash
docker rmi  simulot/immich-go
docker run -t --rm  simulot/immich-go upload <path to upload>

Without a docker image, i can't find a way to do this, if anyone provided a one Liner script to achive this, that would be The One as well :)

Don't take me for serious here, but it is more than doable... 😅

#!/bin/bash

# Exit on any error
set -e

# Find latest version
latest=$(curl -s https://api.github.com/repos/simulot/immich-go/releases/latest | grep tag_name | sed 's/.*"tag_name": "\(.*\)",/\1/')

# Download latest version
curl -L -o /tmp/immich-go_Linux_x86_64.tar.gz "https://github.com/simulot/immich-go/releases/download/$latest/immich-go_Linux_x86_64.tar.gz"

# Extract archive to /tmp
tar -xf /tmp/immich-go_Linux_x86_64.tar.gz -C /tmp/

# Copy binary to /usr/local/bin
sudo cp /tmp/immich-go /usr/local/bin

# Do what you need
immich-go --version

@simulot
Copy link
Owner

simulot commented Jun 23, 2024

Do what you need

immich-go --version

Got it

@pkking pkking closed this Jun 29, 2024
@redge76
Copy link

redge76 commented Sep 12, 2024

@simulot and @pkking
Having a docker image was a great idea. It's totally doable to use @erkexzcx script to update immich-go but it's still a lot more complicated than:
docker pull simulot/immich-go

It's also easier than downloading/installing the binaries.

Docker is not only for 24/7 services. It's also great for simple commands ligne immich-go.
One you start to use docker, it make sense to use it for everything.

Immich CLI has a docker image. (yes with npm it make even more sense to have one...)

Please reconsider the possibility to merge this PR.

@simulot simulot added this to the new CLI milestone Sep 13, 2024
@pkking
Copy link
Author

pkking commented Sep 15, 2024

I am OK to maintain docker stuffs, although i didnot use immich-go for a while.

Also i notice some actions already been added to the project to help, thats awesome, if you guys need docker images, im happy to reopen this PR @simulot @erkexzcx @redge76

@simulot
Copy link
Owner

simulot commented Sep 15, 2024

There are some usages I didn't see, there is maintainer. So let's do it

Thanks for help

@pkking pkking reopened this Sep 15, 2024
Signed-off-by: waaagh <pkwarcraft@gmail.com>
Signed-off-by: lcr <pkwarcraft@gmail.com>
@pkking
Copy link
Author

pkking commented Sep 15, 2024

@simulot please enable the CI, also please add a action secret DOCKERHUB_USERNAME and DOCKERHUB_TOKEN, after that, each time a new tag added, the action will push the image to dockerhub and ghcr as simulot/immich-go

NOTE: i add a new CI to prevent the docker image build failed, each time the golang compiler changed, the ci will complain and people should know Dockerfile need some tweak as well ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants