Skip to content

Conversation

DiSay1
Copy link

@DiSay1 DiSay1 commented Aug 31, 2025

Previously, each service was relying on the default Docker Compose network. As a result, cross-container communication (e.g., transactor → redpanda) failed, causing KafkaJS connection timeouts and WebSocket session errors.

This commit introduces a shared network 'huly_net' and attaches all services to it, allowing reliable service discovery by container names.

The Docker and Docker Compose versions mentioned below are the versions used in the current working environment:

  • Docker version: 28.3.3
  • Docker Compose version: v2.39.1

Changes:

  • All services now use network: huly_net
  • Network is defined at the bottom of the Compose file
  • Container names remain unchanged
  • Healthchecks and ports are untouched

Previously, each service was relying on the default Docker Compose network.
As a result, cross-container communication (e.g., transactor → redpanda)
failed, causing KafkaJS connection timeouts and WebSocket session errors.

This commit introduces a shared network 'huly_net' and attaches all services
to it, allowing reliable service discovery by container names.

The Docker and Docker Compose versions mentioned below are the versions used
in the current working environment:

- Docker version: 28.3.3
- Docker Compose version: v2.39.1

Changes:
- All services now use network: huly_net
- Network is defined at the bottom of the Compose file
- Container names remain unchanged
- Healthchecks and ports are untouched


Signed-off-by: _DiSay <[email protected]>
Copy link

Connected to Huly®: SELFH-196

Copy link
Contributor

@shanzez shanzez left a comment

Choose a reason for hiding this comment

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

I do something just like this for my local instance. The only difference in mine is that I created it as an external net and I have a shared nginx instance because I'm running multiple containers on my system and wanted a single nginx.conf.

@aonnikov
Copy link
Member

aonnikov commented Sep 8, 2025

Thanks for your contribution! Looks good to me, but could you please resolve conflicts when you have a chance?

Copy link
Contributor

@lexiv0re lexiv0re left a comment

Choose a reason for hiding this comment

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

Also, the migration guide to v7 need to be adjusted as a part of this change as it currently assumes use of the default docker network.

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.

4 participants