Skip to content

Commit

Permalink
fix build (#4250)
Browse files Browse the repository at this point in the history
  • Loading branch information
jrhizor committed Jun 21, 2021
1 parent 55d3b4d commit 27ae6f1
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion .env
Expand Up @@ -6,7 +6,7 @@ DATABASE_PASSWORD=docker
DATABASE_HOST=db
DATABASE_PORT=5432
DATABASE_DB=airbyte
DATABASE_URL=jdbc:postgresql://${DATABASE_HOST}:${DATABASE_PORT}/${DATABASE_DB}
DATABASE_URL=jdbc:postgresql://db:5432/airbyte

# When using the airbyte-db via default docker image:
CONFIG_ROOT=/data
Expand Down
4 changes: 2 additions & 2 deletions docker-compose.yaml
Expand Up @@ -120,10 +120,10 @@ services:
- 7233:7233
environment:
- DB=postgresql
- DB_PORT=5432
- DB_PORT=${DATABASE_PORT}
- POSTGRES_USER=${DATABASE_USER}
- POSTGRES_PWD=${DATABASE_PASSWORD}
- POSTGRES_SEEDS=db
- POSTGRES_SEEDS=${DATABASE_HOST}
- DYNAMIC_CONFIG_FILE_PATH=config/dynamicconfig/development.yaml
volumes:
- ./temporal/dynamicconfig:/etc/temporal/config/dynamicconfig
Expand Down
4 changes: 2 additions & 2 deletions docs/operator-guides/configuring-airbyte-db.md
Expand Up @@ -31,10 +31,10 @@ DATABASE_PORT=3000
DATABASE_DB=postgres
```

Additionally, you are free to redefine the JDBC URL constructed in the environment variable `DATABASE_URL` if you need to provide extra arguments to the JDBC driver (for example, to handle SSL):
Additionally, you must redefine the JDBC URL constructed in the environment variable `DATABASE_URL` to include the correct host, port, and database. If you need to provide extra arguments to the JDBC driver (for example, to handle SSL) you should add it here as well:

```bash
DATABASE_URL=jdbc:postgresql://${DATABASE_HOST}:${DATABASE_PORT}/${DATABASE_DB}?ssl=true&sslmode=require
DATABASE_URL=jdbc:postgresql://host.docker.internal:3000/postgres?ssl=true&sslmode=require
```

## Initializing the database
Expand Down

0 comments on commit 27ae6f1

Please sign in to comment.