Skip to content

Conversation

@bobbai00
Copy link
Contributor

@bobbai00 bobbai00 commented Sep 22, 2025

Overview

This PR syncs the docker compose file of the single node architecture with the latest micro service architecture.

Changes

  • Added config-service, computing-unit-managing-service and y-websocket-server to Docker Compose
  • Updated nginx proxy configuration with new service routes
  • Updated all Texera service images to release-1-1-0 tag
  • Fixed service dependencies and health checks
  • Updated environment variables with certain GUI configuration

@bobbai00 bobbai00 self-assigned this Sep 22, 2025
@bobbai00 bobbai00 requested a review from aicam September 22, 2025 06:39
@chenlica
Copy link
Contributor

@aicam Please review this PR.

@bobbai00 bobbai00 force-pushed the release/single-node-1-1-0 branch from a49e4d9 to 22d68cb Compare September 22, 2025 21:06
@bobbai00 bobbai00 force-pushed the release/single-node-1-1-0 branch from d78e705 to 0a19fd3 Compare September 27, 2025 20:24
Copy link
Contributor

@aicam aicam left a comment

Choose a reason for hiding this comment

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

LGTM! I just had two concerns

LAKEFS_STATS_ENABLED=1
LAKEFS_DATABASE_TYPE=postgres
LAKEFS_DATABASE_POSTGRES_CONNECTION_STRING=postgres://texera:password@texera-postgres:5432/texera_lakefs?sslmode=disable
LAKEFS_DATABASE_POSTGRES_CONNECTION_STRING=postgres://texera:password@postgres:5432/texera_lakefs?sslmode=disable
Copy link
Contributor

Choose a reason for hiding this comment

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

Why do we need to remove texera from all variables?

# variables for example user, datasets and workflows
TEXERA_EXAMPLE_USERNAME=texera
TEXERA_EXAMPLE_PASSWORD=texera
TEXERA_EXAMPLE_USERNAME="${USER_SYS_ADMIN_USERNAME}"
Copy link
Contributor

Choose a reason for hiding this comment

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

I think the env should have examples and docker compose override them, here you are using system env inside .env file

@github-actions github-actions bot added the build label Oct 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants