Skip to content

[chore][INF-7329] Allow selectively sending telemetry inputs to Retool#350

Merged
golfdish merged 5 commits into
mainfrom
v/allow-phonehome-metrics-only
Jul 21, 2026
Merged

[chore][INF-7329] Allow selectively sending telemetry inputs to Retool#350
golfdish merged 5 commits into
mainfrom
v/allow-phonehome-metrics-only

Conversation

@golfdish

@golfdish golfdish commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Counterpart to https://github.com/tryretool/retool_development/pull/67405: allow selectively enabling telemetry inputs to be sent to Retool when telemetry.sendToRetool.enabled = true, with only metrics enabled by default.

Note that collecting telemetry is still disabled by default.

@greptile-apps

greptile-apps Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Confidence Score: 5/5

Safe to merge — the change is additive, guarded by a Helm-level validation, and defaults preserve existing behaviour.

The only moving part is a new env var conditionally injected when sendToRetool is enabled. The fail guard prevents misconfiguration, the default value of ["metrics"] keeps current opt-in deployments unaffected, and the template logic is straightforward with no side effects on other chart components.

No files require special attention.

Important Files Changed

Filename Overview
charts/retool/templates/deployment_telemetry.yaml Adds RTEL_SEND_TO_RETOOL_INPUT_ALLOWLIST env var with a Helm-level fail guard preventing empty inputs when sendToRetool is enabled
charts/retool/values.yaml Adds telemetry.sendToRetool.inputs with default ["metrics"] and documents the three available input types
values.yaml Mirror of charts/retool/values.yaml change — identical inputs field added under sendToRetool
charts/retool/Chart.yaml Routine chart version bump from 6.11.11 to 6.11.12

Reviews (5): Last reviewed commit: "Merge branch 'main' into v/allow-phoneho..." | Re-trigger Greptile

Comment thread charts/retool/templates/deployment_telemetry.yaml
@golfdish
golfdish merged commit 98c5ddd into main Jul 21, 2026
13 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.

2 participants