Skip to content

Commit

Permalink
Refactor: Update CI workflow and configuration for PostgreSQL and Cli…
Browse files Browse the repository at this point in the history
…ckHouse ports
  • Loading branch information
Luisotee committed Jan 22, 2025
1 parent a8f6587 commit 29bdd3b
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 5 deletions.
9 changes: 7 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,8 +21,13 @@ jobs:
- name: Install dependencies
run: bun install

- name: Run builds
run: bun run build
- name: Setup config
run: |
cp config.example.yaml config.yaml
bun run build:config
- name: Build remaining packages
run: turbo run build --filter=!@eda/config --filter=!@eda/config-python

# - name: Run tests
# run: bun run test
2 changes: 1 addition & 1 deletion config.example.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ databases:

langtrace_postgres:
host: "langtrace-postgres"
port: 5432
port: 5435
user: "ltuser"
password: "ltpasswd"
database: "langtrace"
Expand Down
4 changes: 2 additions & 2 deletions deploy/langtrace-stack/docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ services:
env_file:
- .env
ports:
- "5432:5432"
- "${POSTGRES_PORT}:5432"
volumes:
- postgres-data:/var/lib/postgresql/data

Expand All @@ -39,7 +39,7 @@ services:
- CLICKHOUSE_USER=${CLICK_HOUSE_USER}
- CLICKHOUSE_DB=${CLICK_HOUSE_DATABASE_NAME}
ports:
- "8123:8123"
- "${CLICKHOUSE_PORT}:8123"
- "9000:9000"
healthcheck:
test: wget --no-verbose --tries=1 --spider http://langtrace-clickhouse:8123/ping || exit 1
Expand Down
2 changes: 2 additions & 0 deletions deploy/langtrace-stack/export-config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@ const envVars = {
POSTGRES_URL_NO_SSL: `postgres://${config.databases.langtrace_postgres.user}:${config.databases.langtrace_postgres.password}@${config.databases.langtrace_postgres.host}/${config.databases.langtrace_postgres.database}`,
POSTGRES_URL_NON_POOLING: `postgres://${config.databases.langtrace_postgres.user}:${config.databases.langtrace_postgres.password}@${config.databases.langtrace_postgres.host}/${config.databases.langtrace_postgres.database}`,
POSTGRES_IMAGE_TAG: "16",
POSTGRES_PORT: config.ports.db.postgres,

// App settings
NEXT_PUBLIC_APP_NAME: config.services.langtrace.api.host,
Expand All @@ -30,6 +31,7 @@ const envVars = {
CLICK_HOUSE_USER: config.databases.langtrace_clickhouse.user,
CLICK_HOUSE_PASSWORD: config.databases.langtrace_clickhouse.password,
CLICK_HOUSE_DATABASE_NAME: config.databases.langtrace_clickhouse.database,
CLICKHOUSE_PORT: config.ports.db.clickhouse,

// Admin settings
ADMIN_EMAIL: config.services.langtrace.admin.email,
Expand Down

0 comments on commit 29bdd3b

Please sign in to comment.