Skip to content

fix: replace SERVICES with TEMPORAL_SERVICES and add service validation#141

Merged
chaptersix merged 3 commits intomainfrom
fix/use-temporal-services-env-var
Apr 10, 2026
Merged

fix: replace SERVICES with TEMPORAL_SERVICES and add service validation#141
chaptersix merged 3 commits intomainfrom
fix/use-temporal-services-env-var

Conversation

@chaptersix
Copy link
Copy Markdown
Contributor

@chaptersix chaptersix commented Apr 9, 2026

Summary

  • Replace deprecated SERVICES env var with TEMPORAL_SERVICES in compose/docker-compose-multirole.yaml and tls/tls-full/docker-compose.yml
  • Add CI validation to the multirole test job that checks each container is running only its expected service role and that ringpop membership is correctly wired on the internal network

Validation added

Service role isolation — uses /proc/net/tcp inside each container to verify:

  • Each service is listening on its expected gRPC port (history→7234, matching→7235, frontend→7237, frontend2→7236)
  • No container is binding a port belonging to a different service role

Ringpop membership reachability — for history (6934) and matching (6935):

  • Confirms the membership port is listening inside the container
  • Confirms the port is reachable cross-container from temporal-admin-tools on temporal-network, verifying ringpop is bound to the right network interface and not just loopback

Update docker-compose-multirole.yaml and tls/tls-full/docker-compose.yml
to use the TEMPORAL_SERVICES env var instead of the deprecated SERVICES.

Add CI validation to the multirole test job that checks each container
is listening on its expected service port and not on any other service's
port, using ss -tln inside each container.
@chaptersix chaptersix changed the title fix: replace SERVICES with TEMPORAL_SERVICES in compose examples fix: replace SERVICES with TEMPORAL_SERVICES and add service validation Apr 10, 2026
@chaptersix chaptersix marked this pull request as ready for review April 10, 2026 12:51
@chaptersix chaptersix requested a review from a team as a code owner April 10, 2026 12:51
@chaptersix chaptersix merged commit efb48a1 into main Apr 10, 2026
22 of 23 checks passed
@chaptersix chaptersix deleted the fix/use-temporal-services-env-var branch April 10, 2026 12:52
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.

1 participant