Skip to content

feat(clientsv2): add option to disable external telemetry#344

Open
xjerod wants to merge 1 commit intomainfrom
jerod/external-telemetry
Open

feat(clientsv2): add option to disable external telemetry#344
xjerod wants to merge 1 commit intomainfrom
jerod/external-telemetry

Conversation

@xjerod
Copy link
Copy Markdown
Contributor

@xjerod xjerod commented Apr 16, 2026

Adds a values flag to disable external telemetry (Grafana Faro) calls from the unionconsole frontend. Needed for egress-restricted selfhosted/selfmanaged deployments where all outbound traffic is blocked.

Default is true (unchanged behavior). Backward compatible — downstream values files that omit the new telemetry block render byte-identical output.

Changes

  • charts/controlplane/values.yaml — add console.telemetry.externalEnabled: true default with inline doc.
  • charts/controlplane/templates/console/deployment.yaml — when externalEnabled is explicitly set to false, append DISABLE_EXTERNAL_TELEMETRY=true to console env. Uses hasKey + explicit false check rather than | default true, which mis-handles false-as-no-value.

@aviator-app
Copy link
Copy Markdown
Contributor

aviator-app Bot commented Apr 16, 2026

Current Aviator status

Aviator will automatically update this comment as the status of the PR changes.
Comment /aviator refresh to force Aviator to re-examine your PR (or learn about other /aviator commands).

This pull request is currently open (not queued).

How to merge

To merge this PR, comment /aviator merge or add the mergequeue label.


See the real-time status of this PR on the Aviator webapp.
Use the Aviator Chrome Extension to see the status of your PR within GitHub.

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