-
-
Notifications
You must be signed in to change notification settings - Fork 5
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: Arm64 images #52
Comments
Hey @aleksasiriski, great to hear from you! Yep diesel is great to allow multiple database backends in Rust, makes development a lot easier. As for arm64, I do have to check out how to do that correctly as I had some issues with building binaries before. As for multiple replicas- the only state that remains in-memory is the timers which I'll be refactoring to the database in #47. The only other issue I see is with the job loops which, in a rare case, could try to process simultaneously. Possibly a solution would be to add a "processing" state to prevent other replicas from touching those ScanEvents, however I'm open to any suggestions! |
Great to hear! Yeah, probably some lock mechanism as you described with "processing" state, just have to think about how to release it if it doesn't get processed (pod crashed or something). So maybe something with TTL that gets it's TTL reset every few seconds by the worker that is processing it. Not sure if you should introduce more complexity with some queueing system component or DIY it using the database. Either way, if you leave it as is currently it's also very good :D |
Hey @aleksasiriski sorry for the long reply! I had some.. difficulty.. with arm64 however I seem to have figured it out with #56 and |
Thanks for your hard work, I will test it next week and get back to you :) |
No problem, I will say using qemu on github actions takes like over an hour to build an arm image, lemme know if you have any ideas on how to make that better |
Couldn't you crossbuild on normal Github CI? That's what I do for Go projects, just set the output to be for arm. Although, then you must build outside of Dockerfile and copy it over inside. |
Cross-building diesel is a bit of a pain sadly, libpq and libsqlite are both temperamental about being statically linked let-alone cross-built. I'll see if doing it outside docker gives me a bit more leeway though |
I've successfully started the container in my K8s cluster, will let you know if it works as expected with sonarr+radarr+jellyfin. |
Hey @aleksasiriski I changed it today to use |
Is there a new tag, or is it already on latest? |
Should be on |
good work on autopulse 👍 , btw will there be arm64 version of autopulse-ui ? |
I tested this and it works good, so you can merge and close this whenever you feel it's good |
Shouldn't be too hard, I'll merge #56 first and then check that out |
@myhme could you check |
I can run the container but not able to login
but i can connect with the autopulse container with the ip so this issue would be similar to #85. btw autopulse:2875 woul not work in my case since i am defining network in compose
below is my compose file
below is my config file
|
Awesome, I'll go ahead and close this |
Description
Hello there, I'm the author of Autoscan continuation after the original author stopped working on it. I also stopped improving it after adding Postgres and would definitely prefer not to have to maintain it if this ends up working good :) I see you also support Postgres (which is great).
As for this issue, I see on Dockerhub that there are no arm64 build of the image. It would be great if you added these (as this is written in Rust, there shouldn't be any problems building cross platform binaries). If you decide to add arm64 images, I will be able to test this on my k8s cluster and see how it works.
Additional question: Have you thought about having multiple replicas of this project running at the same time, is there any kind of queue for the tasks? This would bring load balancing and high-availability to those of us that try to have it with Jellyfin
Additional Information
No response
The text was updated successfully, but these errors were encountered: