Skip to content

[feat] Set R2_ORCHESTRATION_BACKEND=postgres when Temporal is disabled - #368

Merged
westrik merged 2 commits into
mainfrom
r2_pg
Sep 1, 2026
Merged

[feat] Set R2_ORCHESTRATION_BACKEND=postgres when Temporal is disabled#368
westrik merged 2 commits into
mainfrom
r2_pg

Conversation

@westrik

@westrik westrik commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

R² sandbox workflows run on either Temporal or pg-boss. When no Temporal cluster is enabled (neither the retool-temporal-services-helm subchart nor an external temporal config), emit R2_ORCHESTRATION_BACKEND=postgres on the backend, workflow backend, workers, and jobs runner so R² falls back to pg-boss instead of failing to reach a nonexistent Temporal frontend.

Gated on image tag >= 4.47.0 — older images don't support the postgres backend and keep the Temporal default.

R² sandbox workflows run on either Temporal or pg-boss. When no Temporal
cluster is enabled (neither the retool-temporal-services-helm subchart nor
an external temporal config), emit `R2_ORCHESTRATION_BACKEND=postgres` on the
backend, workflow backend, workers, and jobs runner so R² falls back to
pg-boss instead of failing to reach a nonexistent Temporal frontend.

Gated on image tag >= 4.47.0 — older images don't support the postgres backend and keep the Temporal default.
@greptile-apps

greptile-apps Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Confidence Score: 5/5

The PR appears safe to merge because no blocking failure remains within the eligible follow-up scope.

No blocking failure remains.

Important Files Changed

Filename Overview
charts/retool/templates/_helpers.tpl Adds centralized Temporal detection, version-gated R² backend selection, and shared environment propagation.
charts/retool/templates/deployment_jobs.yaml Adds the conditional R² orchestration backend environment entry to the jobs-runner configuration.
charts/retool/Chart.yaml Bumps the chart patch version to publish the template behavior change.

Reviews (2): Last reviewed commit: "Bump chart version to 6.11.24" | Re-trigger Greptile

Comment thread charts/retool/templates/_helpers.tpl
@westrik westrik changed the title Set R2_ORCHESTRATION_BACKEND=postgres when Temporal is disabled [feat] Set R2_ORCHESTRATION_BACKEND=postgres when Temporal is disabled Sep 1, 2026
@westrik
westrik merged commit 1793408 into main Sep 1, 2026
14 checks passed
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