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
[Feature Request] Consider offering an official Docker image #1093
[Feature Request] Consider offering an official Docker image #1093
Comments
I think there might be simpler ways to do this, unless I'm mistaken. Instead of building an image, pulling the jar and running the openjdk image would be way faster to setup (imho), or as an I created the following:
And the docker-compose.yml goes like this : ---
version: "3"
services:
musicbot:
image: openjdk
container_name: musicbot
volumes:
- ./data:/usr/src/myapp
working_dir: /usr/src/myapp
restart: unless-stopped
entrypoint: java
command: "-Dnogui=true -jar JMusicBot-current.jar" Doing that allows the last openjdk version to be pulled and used, and when a new version is released, just replace the symlink in the data folder, then restart the container and everything runs smoothly. |
That does work, but it prevents systems such as Watchtower from working, which automatically pull new image versions. |
True, but as the bot notifies you already about a new version, I just DL the new jar, replace the symlink and restart the container, which seems reasonable compared to creating github jobs etc... for people who might not be as familiar with such processes :) |
I think both approaches should be considered tbh. If people want automated updates with a current image, then your idea is better, if they don't care about updating manually, knowing that the bot updates might be quite rare, then maybe I should create a PR for documenting my method? |
If there will be a docker image for this bot then I'm not sure why one would choose your method, using the bot image is probably easier to set up even if you don't care about updating it. |
I do have Docker Hub Pro, which (I believe) allows me to configure an autobuild to run everytime this repo is updated. I am the developer of the docker "azamserver/pihole-doh" which has 1.5K pulls, and I would be glad to help out. |
Any update on this issue? I'm holding off updating my bots till this is resolved. |
@AzamServer The issue is on-hold, which means that the dev probably has an eye on it but isn't going to move with the changes too quickly. If you want, you can use the images I publish to Docker Hub (https://hub.docker.com/r/alexandreteles/jmusicbot), built and published thru my repository (https://github.com/alexandreteles/jmusicbot_docker). Images are periodically rebuilt based on updates to the base image(s) or to this repository. |
@alexandreteles Thanks for the help, and I will be using your docker! |
@alexandreteles thanks so much for this! |
Any update on this? Will there be an official image? |
Is your feature request related to a problem? Please describe.
Not directly linked to JMusicBot's code, no. Considering that there's a bunch of different Docker images available on DockerHub, the most popular of them having 50K downloads, but it being updated a year ago (!), having an official image for JMusicBot could provide a safer and up to date way for users to run the bot using Docker or Podman.
What is your ideal solution to the problem?
playlistsFolder
location in theirconfig,txt
relative to the/app
path (ex.:playlistsFolder = "/app/playlists"
) and bind a local playlist folder to the bot container:How would this feature be used?
Users wanting to host JMusicBot using Docker or Portainer would have easy of mind that the image they're running is secure. Additionally, users without much experience on server administration could easily host the bot with a single command.
Additional Info
All the steps mentioned above, with some changes, are being used on my repository (https://github.com/alexandreteles/jmusicbot_docker) to build and publish a Docker image (https://hub.docker.com/r/alexandreteles/jmusicbot) on those parameters. Please, go check it out if you have any questions on how the process works. Especial detail for the Security tab being populated with the relevant information about vulnerabilities in the image.
The code is under the WTFPL so, well, "Do What The F*ck You Want".
In any case, having it built and shipped by the project maintainer would make it much more trusted, especially if the build process is transparent and mentioned in the image description. Even if it should be just a bit of CTRL+C/CTRL+V for it to be setup, I could submit the necessary PRs. You would still need to setup the secrets, tho.
Checklist
The text was updated successfully, but these errors were encountered: