Skip to content

docs: document CONNECTOR_DISCOVERY_ENABLED for Fusion-only / air-gapped deployments#150

Open
antoniocali wants to merge 2 commits into
datazip-inc:stagingfrom
antoniocali:docs/connector-discovery-flag
Open

docs: document CONNECTOR_DISCOVERY_ENABLED for Fusion-only / air-gapped deployments#150
antoniocali wants to merge 2 commits into
datazip-inc:stagingfrom
antoniocali:docs/connector-discovery-flag

Conversation

@antoniocali

Copy link
Copy Markdown
Contributor

Summary

Companion to datazip-inc/olake-ui#382, which adds a CONNECTOR_DISCOVERY_ENABLED server flag (and a matching DEFAULT_CONNECTOR_VERSION) to skip Docker Hub / ECR / GCR queries in Fusion-only and air-gapped deployments (datazip-inc/olake-ui#380).

The chart needs no template change to support this — olakeUI.env is already a passthrough map, so any env var set there is propagated to the UI pod. This PR is purely about discoverability: putting the new knob where operators will actually see it.

Changes

  • helm/olake/values.yaml — add a commented example under olakeUI.env showing how to disable discovery and pin a default version.
  • helm/olake/README.md — add a "Disabling Connector Discovery (Fusion-only / Air-gapped)" subsection alongside the existing "Private Container Registry" section. Explains the use case (Fusion-only Iceberg maintenance, no CDC) and the registry-rate-limit motivation.

Notes

Test plan

  • helm lint helm/olake/ clean
  • Manual: render with helm template ... --set olakeUI.env.CONNECTOR_DISCOVERY_ENABLED=false --set olakeUI.env.DEFAULT_CONNECTOR_VERSION=v0.3.18 and confirm both vars appear in the olake-ui Deployment env (they should, since olakeUI.env is already passed through via the existing range loop)

vikaxsh and others added 2 commits May 20, 2026 19:13
chore: staging -> master - worker v0.3.6
Surface the new olake-ui config flag through the chart's values.yaml comments
and README. No template change is needed since olakeUI.env is already a
passthrough; this is purely a discoverability improvement so operators of
Fusion-only / air-gapped deployments can find the knob.

Companion to datazip-inc/olake-ui#382 and datazip-inc/olake-ui#380.
@schitizsharma

Copy link
Copy Markdown
Collaborator

Hi @antoniocali could you please take a pull from the latest staging branch to fix the vulnerabilities.

Is this feature urgent? We are discussing internally to fix this on both worker and ui side currently.

@antoniocali

Copy link
Copy Markdown
Contributor Author

Hi @antoniocali could you please take a pull from the latest staging branch to fix the vulnerabilities.

Is this feature urgent? We are discussing internally to fix this on both worker and ui side currently.

Not urgent
Things works, just create a lot of noise

Let me anyway merge from staging and push to resolve conflicts

@schitizsharma schitizsharma changed the base branch from master to staging June 2, 2026 09:16
@antoniocali

Copy link
Copy Markdown
Contributor Author

Hi @antoniocali could you please take a pull from the latest staging branch to fix the vulnerabilities.

Is this feature urgent? We are discussing internally to fix this on both worker and ui side currently.

Just to also check, this PRs would make sense on helm only after datazip-inc/olake-ui#382 is closed, since that's where the change for connector-discovery-flag is implemented.

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