search

MrLoLf / HoneygainAutoClaim

Honeygain Auto Claimer bot for lucky pot, achievements and referrals every day via GitHub actions, python script or Docker file
MIT License
42 stars 86 forks source link

[FR] Docker and Python upgrade #19

Open madereddy opened 8 months ago

madereddy commented 8 months ago

Automating Docker builds and updating the python script to allow Honeygain creds and cron expression to be stored as env variables for the container, this would allow someone to run both Honeygain and autoclaim from the same machine.

MrLoLf commented 8 months ago

The Docker image can now be run with env arguments.

MrLoLf commented 8 months ago

There is now a docker image, but I can't automate it, due to docker hub wanting $5 per month from me to have automatic builds. Just the cron job is left, I'll look into this in the near future.

madereddy commented 8 months ago

I tested the docker image and the env variables work great. You can use Github actions to build new docker images for free and push to DockerHub. I will push a commit over with the workflow steps commented out so you can review it as well. You will need to put in your DockerHub user and password into Github secrets for it to work.

MrLoLf commented 8 months ago

I've done this already, but I probably made something wrong, maybe you'll be able to spot it. I've added this secretes, you can find the file responsible for it in the .github folder it's called Dockerfile.yaml if I'm not mistaking. I currently have pushed the image manually to the docker hub repo. Is it possible to have this image for not only amd64 like also arm64 and so on?

madereddy commented 8 months ago

I think you need to have the Dockerfile.yaml moved one folder deeper into the workflows folder for it to work. Can you try moving that over and see if that works automatically? I am currently using the method in https://github.com/MrLoLf/HoneygainAutoClaim/pull/31 to push both AMD64 and ARM64 images. Those are the only images that the base ChainGuard image supports. If you want to support other images, it would require a different base image.

MrLoLf commented 7 months ago

I have removed it, as I have merged your PR. I've also added a manual trigger. It works now, thanks a lot. It would be maybe nice for the future to support also RISC-V, as it's getting better and better and more popular.