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

Add support for suspend #2290

Open
ChenYi015 opened this issue Oct 28, 2024 · 3 comments · May be fixed by #2387
Open

Add support for suspend #2290

ChenYi015 opened this issue Oct 28, 2024 · 3 comments · May be fixed by #2387
Assignees

Comments

@ChenYi015
Copy link
Contributor

What feature you would like to be added?

  • Add a new field .spec.suspend to the SparkApplication CRD.
  • The controller will not submit the SparkApplication except for the .spec.suspend is false.

Why is this needed?

No response

Describe the solution you would like

No response

Describe alternatives you have considered

No response

Additional context

No response

Love this feature?

Give it a 👍 We prioritize the features with most 👍

@tariq-hasan
Copy link

/assign

@munali
Copy link

munali commented Jan 14, 2025

@tariq-hasan - curious if you still plan to implement this for Spark Operator? We are looking into using Kueue for Spark Applications and this feature is a prerequisite.

@everpeace everpeace linked a pull request Jan 17, 2025 that will close this issue
8 tasks
@everpeace
Copy link

everpeace commented Jan 17, 2025

@tariq-hasan I also need to integrate SparkApplication into Kueue. As @munali wrote, suspend/resume feature would enable seamless integration with Kueue.

Would you mind me taking over this issue from you? Actually my PR(#2387) has been already pushed. I'm really appreciated if you gave a review to my PR, too.

Moreover, I'm working on Kueue integraiton on my personal branch: https://github.com/everpeace/kueue/tree/kubeflow-spark-operator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants