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

fix: Dockerfile #40

Merged
merged 1 commit into from
Dec 9, 2024
Merged

Conversation

codingluke
Copy link
Contributor

@codingluke codingluke commented Dec 2, 2024

Liebe Gruppe Insimodus

nach langem ausprobieren konnte ich die App nach AWS deployen.

Es scheint so als ob das offizielle nginx Image eine andere Standardkonfiguration besitzt als der manuell installierte von meinem Beispiel. Auf jeden Fall klappt das Deployment mit meinem Beispiel und eurem Code (siehe Dockerfile)

Natürlich wäre es eleganter direkt das offizielle nginx image zu nehmen. Wie man vorgehen könnte um dies genauer zu analysieren wäre folgendermassen:

  1. Lokal ein manueller Nginx und der Offizielle Nginx als Container starten.
  2. Von Beiden die Standardkonfig zum Host kopieren: docker run --rm --entrypoint=cat nginx /etc/nginx/nginx.conf > /host/path/nginx.conf
  3. Prüfen was sich genau unterscheidet.

Auf jeden Fall habt ihr keine "Schuld", dass es nicht geklappt hat. Eigentlich sollte es auch mit dem offiziellen nginx image gehen.

Beste Grüsse
Lukas Hodel

@@ -1,15 +1,15 @@
server {
listen 3000 default_server;
listen [::]:3000 default_server;

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nur Formattierung.

@codingluke codingluke requested a review from lev1nn December 3, 2024 06:35
@codingluke
Copy link
Contributor Author

codingluke commented Dec 3, 2024

@lev1nn der PR muss vor #39 gemerged werden, er zeigt auf ihren branch.

edit: Nachtrag zu den Standardconfigs, ich habe mal beide verglichen, die vom offiziellen image und die vom manuell installierten und konnte kein Unterschied feststellen. Magie gibt es nur in Computerspielen 😅 Da muss irgend wo ein Detail sein...

Nachtrag2: Es ist zu 90% der Port 80 der vom nginx Image exposed wird und mit dem Proxy kollidiert. Ich suche noch nach der kamal konfig dies zu beheben. Bis dahin bitte, wie in dem PR, nginx manuell bauen.

Screenshot_2024-12-03_001158

@robinb19 robinb19 merged commit ae91f11 into bbzblit:feat/aws-deploy Dec 9, 2024
1 check failed
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.

2 participants