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

Προσθήκη απαραίτητων αρχείων για γρήγορη εγκατάσταση και εκτέλεση με Docker #14

Merged
merged 4 commits into from Apr 30, 2023

Conversation

Sugarv
Copy link
Contributor

@Sugarv Sugarv commented Apr 18, 2023

Με τον τρόπο αυτό η εφαρμογή είναι πλήρως λειτουργική και το μόνο που εγκαθιστούμε στο σύστημα είναι το Docker. Δε χρειάζεται εγκατάσταση Laravel, Web server, βάσης, ούτε παραμετροποίηση αυτών. Συνεπώς, η μέθοδος αυτή προτείνεται και για γρήγορη δοκιμή.

@g-theodoroy
Copy link
Owner

g-theodoroy commented Apr 22, 2023

Καλημέρα Βαγγέλη και Χριστός Ανέστη

Ευχαριστώ για την προσφορά σου. Την αποδέχομαι και θα την καλοδεχτώ.

Έχω την εξής ιδέα - ερώτηση.

Θα μπορούσαμε στον Dockerfile γραμμή 17

αντί για

COPY . /var/www/html

να κάνουμε

RUN git clone ...(κλπ) ?

Με τον τρόπο αυτό το merge θα στέκονταν σαν αυτούσιο repository

όπου θα κάνει εγκατάσταση τραβώντας τα αρχεία του electronic_protocol.

Έτσι θα είναι εύκολη η συντήρηση και των δύο project.

Όπως αντιλαμβάνομαι εστιάζει σε ubuntu.

Θα μπορούσε στο μέλλον να τροποποιηθεί για debian ή centos ή άλλο.

@Sugarv
Copy link
Contributor Author

Sugarv commented Apr 22, 2023

Χαίρομαι που βρήκες χρήσιμο το pull request!

Να σου πω την αλήθεια, δε βρίσκω λόγο να γίνει δεύτερο repo μόνο για docker deployment.
Αν παρατηρήσεις, θα δεις ότι τα περισσότερα OSS projects έχουν μέσα τους τα docker-compose & Dockerfile και όχι σε ξεχωριστό repo.

Επίσης κατηγορηματικά όχι, η προτεινόμενη λύση δεν εστιάζει σε ubuntu. Το docker παίζει παντού το ίδιο, αυτή είναι η αρχή του. Η λύση που προτείνω παίζει άψογα και σε windows με εγκατεστημένο το docker desktop.

Για να το κάνω πιο εύκολο για τους χρήστες του project, πρόσθεσα ακόμη δύο αρχεία, τα docker-install.sh & docker-install.bat για γρήγορο deployment σε υπολογιστές με linux & windows αντίστοιχα (φυσικά με το docker εγκατεστημένο).

Νομίζω ότι οι τελευταίες αλλαγές καλύπτουν το σύνολο των περιπτώσεων.

@g-theodoroy g-theodoroy merged commit f876f34 into g-theodoroy:master Apr 30, 2023
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

2 participants