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] Allow to choose Spot Instances at workflow start time #2284

Closed
2 tasks done
Antaxify opened this issue Mar 24, 2022 · 5 comments
Closed
2 tasks done

[Core feature] Allow to choose Spot Instances at workflow start time #2284

Antaxify opened this issue Mar 24, 2022 · 5 comments
Assignees
Labels
enhancement New feature or request flyteadmin Issue for FlyteAdmin Service ui Admin console user interface
Milestone

Comments

@Antaxify
Copy link
Contributor

Motivation: Why do you think this is important?

At this point, we can set interruptible=true either at task or workflow level. I would be great if we could set this flag when we start a workflow, similiar to max parallism. This would allow to use the very same workflow as follows:

  • during development / testing, run with interruptible=false to get feedback asap
  • for big production runs where cost efficiency is more important, run with interruptible=true
  • for time critical executions (e.g. an approaching deadline) , run with interruptible=false

This would be a very helpful addition to Flyte. :)

Goal: What should the final outcome look like, ideally?

  • interruptible can be set at workflow start, similar to max parallism
  • can be set when a workflow ist started using flytectl, console, flyte_remove, or api

Describe alternatives you've considered

Registering the same workflow twice, once with interruptible set to true, once set to false

Propose: Link/Inline OR Additional context

No response

Are you sure this issue hasn't been raised already?

  • Yes

Have you read the Code of Conduct?

  • Yes
@Antaxify Antaxify added enhancement New feature or request untriaged This issues has not yet been looked at by the Maintainers labels Mar 24, 2022
@welcome
Copy link

welcome bot commented Mar 24, 2022

Thank you for opening your first issue here! 🛠

@kumare3
Copy link
Contributor

kumare3 commented Mar 24, 2022

Cc @pmahindrakar-oss / @EngHabu

@kumare3 kumare3 added ui Admin console user interface flyteadmin Issue for FlyteAdmin Service and removed untriaged This issues has not yet been looked at by the Maintainers labels Mar 24, 2022
@kumare3 kumare3 added this to the 1.0.1 milestone Mar 24, 2022
@katrogan
Copy link
Contributor

this could be a great candidate for the newly expanded execution config as a matchable resource too

@MorpheusXAUT
Copy link
Contributor

Hi!
As briefly mentioned on Slack, I'd be willing to look into this, feel free to assign the issue to me 😄

@kumare3
Copy link
Contributor

kumare3 commented Apr 8, 2022

Assigned

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 ui Admin console user interface
Projects
None yet
Development

No branches or pull requests

6 participants