Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Core Feature] Add a default MaxParallelism value to the FlyteAdmin config #1532

Closed
katrogan opened this issue Sep 29, 2021 · 1 comment
Closed
Assignees
Labels
enhancement New feature or request flyteadmin Issue for FlyteAdmin Service

Comments

@katrogan
Copy link
Contributor

Motivation: Why do you think this is important?
Workflow executions can be gated by the max number of parallel nodes that can be processed. Specific override support for this MaxParallelism value already exists but not a generic default in the FlyteAdmin config.

Goal: What should the final outcome look like, ideally?
Default MaxParallelism settable in config.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

[Optional] Propose: Link/Inline OR Additional context

  1. Add a MaxParallelism value to one of the config structs here: https://github.com/flyteorg/flyteadmin/tree/master/pkg/runtime/interfaces
  2. Support a sane default and expose the value here: https://github.com/flyteorg/flyteadmin/tree/master/pkg/runtime
  3. Read this value when preparing a request to create a workflow execution here
@katrogan katrogan added enhancement New feature or request flyteadmin Issue for FlyteAdmin Service labels Sep 29, 2021
@kumare3
Copy link
Contributor

kumare3 commented Sep 29, 2021

cc @pmahindrakar-oss can you do this one?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request flyteadmin Issue for FlyteAdmin Service
Projects
None yet
Development

No branches or pull requests

3 participants