Skip to content

Commit

Permalink
Change nginx port
Browse files Browse the repository at this point in the history
Fix conflicts when deploying to Google Cloud Run
  • Loading branch information
hfxbse committed Nov 19, 2024
1 parent ae0f64f commit 1105e5e
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 4 deletions.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ The service can be started through [Docker Compose](https://docs.docker.com/comp
docker-compose up
```

After this, the [user application](http://localhost:8080), [Airflow](http://localhost:8080/airflow/),
After this, the [user application](http://localhost:5000), [Airflow](http://localhost:5000/airflow/),
[Hadoop task overview](http://localhost:8088), and the
[Hadoop file viewer](http://localhost:9870/explorer.html) should be reachable.

Expand Down
3 changes: 2 additions & 1 deletion docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,13 +3,14 @@ services:
build:
context: ./nginx
ports:
- "8080:8080"
- "5000:5000"
depends_on:
- frontend
- airflow
environment:
- AIRFLOW=airflow
- FRONTEND=frontend
- INCLUDE_PORT=on

frontend:
build:
Expand Down
6 changes: 4 additions & 2 deletions nginx/default.conf
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
server {
listen 8080;
listen 5000;

port_in_redirect ${INCLUDE_PORT};

location / {
proxy_pass "http://${FRONTEND}:3000/";
Expand All @@ -13,6 +15,6 @@ server {
proxy_redirect off;

auth_basic "Airflow Admin UI";
auth_basic_user_file /tmp/credentials/airflow;
auth_basic_user_file /credentials/airflow;
}
}

0 comments on commit 1105e5e

Please sign in to comment.