Skip to content
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

[IMP] Set up the system with docker #113

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fkantelberg
Copy link

Hallo @MaMaKow,

für einen Kunden sollte dieses Projekt als Schichtplan aufgesetzt werden. Damit dies reibungsloser in unsere Infrastruktur passt, haben wir uns entschieden dies mit Hilfe von docker/docker-compose umzusetzen. Die Änderungen würden wir gern zurückspielen, da man damit auch schneller mal PHP und Datenbankversionen testen kann.

Dabei sind ebenso ein paar Probleme im PHP aufgetaucht, die wir vorerst mittels eines Patches (fixes.patch) gelöst haben. Den Patch würde ich vor dem Merge wieder entfernen, wenn die Änderungen soweit von dir abgenommen sind.

Leider war es nicht möglich die config.php durch docker zu erstellen, da ich keine schnelle Möglichkeit gefunden habe, das Datenbankschema und den Admin-Nutzer einfach über ein php-Skript zu erstellen. Dies wäre gut, damit man noch einfacher Parametrisieren und Aufsetzen kann, ohne die Datenbank-Verbindung etc. eintragen zu müssen.

@OSevangelist
Copy link

Hallo @MaMaKow leider war das ganze im Test für unseren Anwendungssfall dann doch nicht geeignet, da wir es für eine NICHT-Apotheken Domäne verwenden wollten. Außerdem hat sich gezeigt das das o.g. Role Out meines Kollegen @fkantelberg auch noch das Problem hatte dass zusätzlich noch

SET GLOBAL sql_mode=(SELECT REPLACE(@@sql_mode,'ONLY_FULL_GROUP_BY',''));

Im Datenbankcontainer auszuführen war um Probleme mit der verwendeten GROUP BY Funktion zu eleminieren. Dennoch wäre auch aus meiner Sicht ein Docker-Compose basiertes Setup für das Projekt wünschenswert.

@MaMaKow
Copy link
Owner

MaMaKow commented Nov 27, 2020

Vielen Dank für die Kommentare. Ich werde heute Abend mal drauf schauen, wenn die Kinder im Bett sind.

Mit docker habe ich mich bisher nicht beschäftigt, weil ich primär zusätzliche Komplexität sehe, ohne großen Benefit für den Anwender.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants