feat(Docker): Docker Support

[piechade]

So könnte ein Docker Support aussehen.

vaccipy:
    image: vaccipy
    container_name: vaccipy
    restart: always
    environment:
      - VNC_PASSWORD=<password>
      - VNC_PORT=5902
      - WEB_PORT=6902
    ports:
      - 5902:5902
      - 6902:6902
    volumes: 
      - ./vaccipy/kontaktdaten.json/:/app/kontaktdaten.json

Außerdem ist die novnc zwar da, läuft aber nicht. Ich hab es auf die schnelle mit VNC-Viewer gemacht, was keine Lösung ist. Ich hatte zwei bots laufen und musste deshalb die Port config nach außen holen. Ist das eine gute Idee? Lieber alles auf eine Instanz optimieren?

[timreibe]

chrome_options.add_argument("--no-sandbox") hatten wir mal und bewusst wieder rausgenommen, um Malware zu vermeiden. Ist das zwingend notwendig für Docker?

[piechade]

Eine gute Frage, ich hab das ohne zu hinterfragen immer rein gemacht. Hier ist ein Argument von jemanden anderen.
https://github.com/elgalu/docker-selenium
Docker ist ja eine sandbox und es läuft so und so alles unter root. Ich denke das ist z. B., wenn man unter Windows normal läuft viel wichtiger. Natürlich müsste man eigentlich die Sache etwas modifizieren. Vielleicht einen --docker Parameter der die sandbox reinmacht und ohne Frage die json nimmt.

[sinun98]

Headless funktioniert leider nicht. Kann dann maximal zur Terminsuche genutzt werden. Buchung und Code-Gen benötigt eine GUI.

[timreibe]

warum wird der Pfad hier rausgenommen?

[piechade]

Zeile 16 im Docker File war nicht erfolgreich. Im Docker Container ist der chrome driver schon da und geht ohne den Parameter. Aber ja das muss man besser machen.

[haslersn]

Hast du die Umgebungsvariable VACCIPY_CHROMEDRIVER verwendet?

[piechade]

Hast du die Umgebungsvariable VACCIPY_CHROMEDRIVER verwendet?

Ist ja im Dockerfile prinzipiell drin.

[piechade]

Jetzt bin ich wieder hier, wenn die Zeile wieder die Methode aufruft.

2021-06-10 13:52:46.857   impfterminservice   [err] [*C4Q2 | 68163, 69124, 69469]   (unknown error: DevToolsActivePort file doesn't exist)
(The process started from chrome location /usr/bin/chromium is no longer running, so ChromeDriver is assuming that Chrome has crashed.)

[haslersn]

Newline am Ende der Datei fehlt

Suggested change

	- ./vaccipy/kontaktdaten.json/:/app/kontaktdaten.json
	- ./vaccipy/kontaktdaten.json/:/app/kontaktdaten.json

[haslersn]

Was ist das? Ich glaube wir hatten das schon mal aktiviert. Warum wurde es wieder deaktiviert?

[haslersn]

Ich würde vorschlagen, dass wir den PR ablehnen. Wer das mit Docker verwenden will, kann ein out-of-tree Dockerfile maintainen. Wenn wir den PR annehmen, bekommen wir zusätzliche Issues, wenn das Dockerfile nicht mehr funktioniert.

[piechade]

Ich würde vorschlagen, dass wir den PR ablehnen. Wer das mit Docker verwenden will, kann ein out-of-tree Dockerfile maintainen. Wenn wir den PR annehmen, bekommen wir zusätzliche Issues, wenn das Dockerfile nicht mehr funktioniert.

Bin eigentlich deiner Meinung, hab nur in Slack gesehen, dass Leute Interesse haben. Ich will das auf meinem Server, wo ein Container OS läuft, laufen lassen. Das ist ein sehr persönlicher Fall und ist für den einfachen User nicht zu empfehlen. Falls einer fragt, kann man darauf verweisen.