Skip to content

Add info about env vars for Geneva workers#142

Merged
dantasse merged 2 commits intomainfrom
dantasse/env_vars
Feb 16, 2026
Merged

Add info about env vars for Geneva workers#142
dantasse merged 2 commits intomainfrom
dantasse/env_vars

Conversation

@dantasse
Copy link
Contributor

Fixes https://linear.app/lancedb/issue/GEN-282/docs-show-how-to-send-env-vars-to-workers

Multiple users have had questions about how to send env vars to workers. In addition to https://github.com/lancedb/geneva/pull/552 and https://github.com/lancedb/geneva/pull/551, this will make it easier to figure out how to do so.

.head_group(
image="myregistry.example.com/my-custom-head-image:latest",
)
.add_worker_group(KubeRayClusterBuilder.cpu_worker()
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry, this is not the same thing, but I noticed it while I was here - want to make this tip clear that you can define images for head or workers this way.

Copy link
Contributor

@jmhsieh jmhsieh left a comment

Choose a reason for hiding this comment

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

remove the create_external in the commend and this is good to go


```python Python icon="python"
cluster = (
GenevaCluster.create_kuberay(cluster_name) # or create_external(cluster_name)
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think this works with create_external as the code stands today. Just remove this comment?

Copy link
Contributor

Choose a reason for hiding this comment

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

ah, saw patches to add support with create_external clusters

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah, I'll merge this right after I merge that.

@dantasse dantasse merged commit b4abd35 into main Feb 16, 2026
2 checks passed
@dantasse dantasse deleted the dantasse/env_vars branch February 16, 2026 17:07
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

Comments