Skip to content

Configure local develop env to use Temporal auto-setup for configurability#202

Open
liam-lowe wants to merge 1 commit intomainfrom
liam-lowe/split-temporal-version
Open

Configure local develop env to use Temporal auto-setup for configurability#202
liam-lowe wants to merge 1 commit intomainfrom
liam-lowe/split-temporal-version

Conversation

@liam-lowe
Copy link
Contributor

@liam-lowe liam-lowe commented Mar 10, 2026

What was changed

  • The development docker compose now uses temporalio/auto-setup instead of temporalio/temporal dev server.
  • The development docker compose is split into multiple files, which are merged at exec time.

Why?

  • The proxy supports connectivity between Temporal clusters with different configuration (Temporal Server Version, Visibility Store, Persistence Store). Using the temporalio/temporal dev server didn't give configurability. This setup will facilitate more robust local manual testing

Comment on lines +2 to +3
TEMPORAL_A_VERSION=1.29.4
TEMPORAL_B_VERSION=1.22.7
Copy link
Contributor Author

Choose a reason for hiding this comment

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

example values - but these can connect ! (didn't validate a full replica)

# Infrastructure versions
# TODO: currently shared by both servers, this could be independent.
POSTGRESQL_VERSION=16
ELASTICSEARCH_VERSION=7.17.27
Copy link
Contributor Author

Choose a reason for hiding this comment

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

In future - could make this more configurable.

I think I'm avoiding making this too comprehensive. We'll need a more-robust test harness for validating version connectivity - and I don't think docker compose and shell scripts are the solution for that.

As a followup, I'll look into adding a testcontainers harness which iterates through Temporal Server versions (and later backend configurations) to validate different functionality (connectivity, unidirectional replica, bidirectional replica, etc)

@echo 'Temporal server versions:'
@printf ' %-16s' 'temporal-a:' && \
$(DOCKER_COMPOSE) exec -T temporal-a \
sh -c 'temporal operator cluster describe --address $$(hostname -i):7233 -o json 2>/dev/null | awk -F\" "/(serverVersion|server_version)/{print \$$4}"' \
Copy link
Contributor Author

Choose a reason for hiding this comment

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

serverVersion|server_version

For Temporal Server compatibility.

@liam-lowe liam-lowe marked this pull request as ready for review March 10, 2026 19:24
@liam-lowe liam-lowe requested a review from a team as a code owner March 10, 2026 19:24
@robholland
Copy link

Sorry to be the bearer of bad news, but auto-setup images are deprecated now. Best not to switch to them.

@temporal-nick
Copy link
Collaborator

Bummer!

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.

3 participants