Dockerfile improvement/fix

Pheromir commented 3 years ago

Hey,

Ich war mal so frei das Dockerimage zu überarbeiten, da dies bei mir aufgrund mehrerer Fehler überhaupt nicht laufen wollte:

Einige shared Libraries haben gefehlt
Geckodriver konnte nicht gefunden werden (in der anibot.py wird Firefox bei Docker enforced)
..Firefox konnte nicht gefunden werden Außerdem habe ich das Base-Image auf ein Slim-Image umgestellt, damit es nicht wesentlich größer wird trotz der ganzen zusätzlichen Dependencies. Da die anibot.py als CMD und nicht als Entrypoint ausgeführt wurde, haben außerdem die Befehle zum Config editieren (add / edit / remove) aus der Readme nicht funktioniert, weil damit der gesamte Befehl "py anibot..." überschrieben und somit versucht wurde den Shell-Befehl "add" etc. auszuführen. Dies ist nun ebenfalls angepasst.

Ebenfalls habe ich die chromedriver-zip entfernt, da die Binary dazu sowieso im Repo liegt und somit nur redundant ist.

Weitere Überlegungen die ich nicht umgesetzt habe, da ich nicht genau weiß wie du das Handhaben möchtest:

Wäre es eventuell sinnvoller, die Chromedriver-Binary komplett aus dem Repository zu entfernen, da sie sonst jeder mit klonen "muss", selbst wenn Firefox mit Geckodriver genutzt werden soll? (Das könnte das Docker-Image nochmal um 10 MB verkleinern :D)
Sollte der Ordner __pycache__ eventuell ebenfalls entfernt werden, da er keinen wirklichen Mehrwert hat?

Pheromir commented 3 years ago

Habe nochmal etwas nachgearbeitet, so dass die config wieder in /config statt /usr/src/app/config gemounted werden kann, die Dockerfiles (da noch eine improvisierte für ARM7 dazu gekommen ist) und die docker-compose.yml in ein Unterverzeichnis (./docker) verschoben, damit nicht das Stammverzeichnis so vollgeklatscht ist.

(Habe leider auch vorhin erst die anderen Dockerfiles von dir entdeckt, in der der CMD korrekt mit --docker, --configfile etc aufgerufen werden >_>)

Pheromir commented 3 years ago

Das überarbeitete arm7 Image sollte auch soweit funktionieren, so dass es Issue #12 löst

Pfuenzle commented 2 years ago

Hi, vielen Dank für die Pullrequest, sieht gut aus. zu dem chromedriver bzw binaries, die habe ich ausversehen mal gepusht und seither nicht mehr gelöscht. Vielen Dank für das Dockerfile, habe da nicht allzuviel Zeit reingesteckt, deins sieht besser aus :D

Pfuenzle / anime-loads

Dockerfile improvement/fix #16