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

Sync kubemark patches into volcano #109

Closed
TommyLike opened this issue Apr 28, 2019 · 5 comments
Closed

Sync kubemark patches into volcano #109

TommyLike opened this issue Apr 28, 2019 · 5 comments
Labels
kind/feature Categorizes issue or PR as related to a new feature. priority/low

Comments

@TommyLike
Copy link
Contributor

TommyLike commented Apr 28, 2019

Is this a BUG REPORT or FEATURE REQUEST?:

/kind feature

There are two patches in kube-batch which are introducing kubemark into kube-batch, we need sync them into volcano as well.

  1. Fix kubemark issue & Add document for kubemark test kubernetes-retired/kube-batch#817
  2. Add kubemark support kubernetes-retired/kube-batch#810

Have to mention that the testcase should be upgraded since we don't the original kubernete Job resource in test.

@TommyLike
Copy link
Contributor Author

/priority low

@TommyLike
Copy link
Contributor Author

@asifdxtreme would you please help on this?
/cc @k82cn

@k82cn
Copy link
Member

k82cn commented Apr 28, 2019

/kind feature

@volcano-sh-bot volcano-sh-bot added the kind/feature Categorizes issue or PR as related to a new feature. label Apr 28, 2019
@TommyLike
Copy link
Contributor Author

@k82cn do we still need this now as we already forked the kube-batch into volcano org?

@k82cn
Copy link
Member

k82cn commented Apr 28, 2019

oh, it's not necessary; but we need to run it in each release against kube-batch:volcano-master branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature. priority/low
Projects
None yet
Development

No branches or pull requests

3 participants