Skip to content

Add worker profiles#353

Draft
THardy98 wants to merge 9 commits into
mainfrom
explore-worker-configs
Draft

Add worker profiles#353
THardy98 wants to merge 9 commits into
mainfrom
explore-worker-configs

Conversation

@THardy98
Copy link
Copy Markdown
Contributor

What was changed

This change adds the ability to specify a --worker-profile in the Omes CLI, a string key that selects a worker configuration specified in code.

The value is passed as an env var to the language worker processes. I chose an environment variable to avoid having to add a flag to each worker's CLI parser (it's a string that doesn't need parsing and I think we want to minimize the parser/flag plumbing code).

Why?

This pattern avoids having to create worker option flags for each worker knob we want to expose. Instead, we can specify these programmatically, as was already possible with the SDK.

The trade-off is that worker configuration lives in code, changing the configuration of a worker requires creating a new profile (which then can be switched at call time by specifying that new profile).

In the future, we may consider adding something akin to --worker-config-json flag to Omes, where users could specify arbitrary JSON to customize worker options in their test/project. That way worker-profile allows for shared worker configs between tests/projects, while --worker-config-json provides flexible, test/project-specific configuration.

Checklist

  1. Closes

  2. How was this tested:

  1. Any docs updates needed?

@THardy98 THardy98 force-pushed the explore-worker-configs branch from bea8e80 to 420af6f Compare May 19, 2026 14:26
@THardy98 THardy98 marked this pull request as ready for review May 19, 2026 14:27
@THardy98 THardy98 requested review from a team as code owners May 19, 2026 14:27
@THardy98 THardy98 force-pushed the explore-worker-configs branch from 45dc169 to 170333d Compare May 28, 2026 14:51
@THardy98 THardy98 marked this pull request as draft May 28, 2026 14:51
@THardy98 THardy98 force-pushed the explore-worker-configs branch from 08aa7f8 to 3d23b3c Compare May 29, 2026 18:19
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