Akkudoktor-EOS / EOS

Other
11 stars 5 forks source link

Setup containerisieren #7

Closed njoerd114 closed 2 days ago

njoerd114 commented 4 days ago

Es wäre wünschenswert, das Projekt als Docker-Image zur Verfügung zu stellen und eine Anleitung (bspw. per docker-compose.yaml) zu geben, um das Projekt inkl. Abhängigkeiten Plattform-Unabhängig zu starten.

dsiebel commented 3 days ago

@njoerd114 @drbacke kann ich mich gerne drum kümmern

danimo commented 3 days ago

@dsiebel Super! Schicke gerne einen PR. Ich kann falls nötig noch Dinge in den Projekt-Settings freischalten.

njoerd114 commented 3 days ago

ich fänd es wichtig, dass wir uns dazu etwas abstimmen, kriterien wären für mich:

dsiebel commented 3 days ago
  • multi-platform

Hatte in erster Linie vor das ganze in eine docker-compose.yaml zu kippen. Multi-arch wäre dadurch gegeben, dass das ganze da gebaut wird wo es auch läuft: docker compose up --build. Multi-arch können wir gerne nochmal drüber reden, wenn die images auch veröffentlicht werden sollen, z.B. über GitHub packages.

  • distroless?

Nicht unmöglich, aber reichlich schwierig umzusetzen für Python mit den ganzen dependencies hier.

PR is allerdings schon fertig, habe die Kommentare hier erst jetzt gesehen. https://github.com/Akkudoktor-EOS/EOS/pull/11

Konnte noch nicht verifizieren, dass das ganze so auch funktioniert, API-wise.

danimo commented 3 days ago

Ich denke das docker-compose.yml ist für den ersten Aufschlag ok. Sobald wir Releases bauen, können wir fertige Pakete, auch multi-Platform, per CI bauen lassen und publisen.

dsiebel commented 2 days ago

Ich denke, das hier kann nun geschlossen werden? #11

danimo commented 2 days ago

Ja, die MacOS Port-Geschichte sollten wir nicht in Docker lösen, sondern durchgehnd. Dafür mache ich ein separates issue auf.