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 unrebased codes #113

Closed
wants to merge 225 commits into from
Closed

Sync unrebased codes #113

wants to merge 225 commits into from

Conversation

TommyLike
Copy link
Contributor

@TommyLike TommyLike commented Apr 29, 2019

Changes are below:

  1. Add help message for command line.
  2. Add enqueue state.
  3. Add Retry support
  4. Enhance PodGroup support.

NOTE: It's depends on the feature in kube-batch, should not be merged before that

The synced codes has not been tested, the e2e testcases would fail due to this.

For #112

k82cn added 30 commits January 8, 2019 09:55
Signed-off-by: Da K. Ma <klaus1982.cn@gmail.com>
Signed-off-by: Da K. Ma <klaus1982.cn@gmail.com>
Signed-off-by: Da K. Ma <klaus1982.cn@gmail.com>
Signed-off-by: Da K. Ma <klaus1982.cn@gmail.com>
Added lifecycle policy framework.
Signed-off-by: Da K. Ma <klaus1982.cn@gmail.com>
Replaced condition by state.
Signed-off-by: Da K. Ma <klaus1982.cn@gmail.com>
Signed-off-by: Da K. Ma <klaus1982.cn@gmail.com>
Signed-off-by: Da K. Ma <klaus1982.cn@gmail.com>
Signed-off-by: Da K. Ma <klaus1982.cn@gmail.com>
Signed-off-by: Da K. Ma <klaus1982.cn@gmail.com>
Signed-off-by: Da K. Ma <klaus1982.cn@gmail.com>
Job management perf enhancement.
Signed-off-by: Da K. Ma <klaus1982.cn@gmail.com>
Signed-off-by: Da K. Ma <klaus1982.cn@gmail.com>
Signed-off-by: Da K. Ma <klaus1982.cn@gmail.com>
Use scheduler name from Job.
Signed-off-by: Da K. Ma <klaus1982.cn@gmail.com>
pkg/admission/admission_controller.go Outdated Show resolved Hide resolved
pkg/apis/batch/v1alpha1/job.go Outdated Show resolved Hide resolved
if _, err := kubeClients.CoreV1().ConfigMaps(job.Namespace).Create(cm); err != nil {
glog.V(3).Infof("Failed to create ConfigMap for Job <%s/%s>: %v",
cmOld.Data = data
if _, err := kubeClients.CoreV1().ConfigMaps(job.Namespace).Update(cmOld); err != nil {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a case we need to update configmap?

pkg/apis/helpers/helpers.go Outdated Show resolved Hide resolved
@@ -128,24 +140,23 @@ func NewJobController(config *rest.Config) *Controller {
cc.cmdLister = cc.cmdInformer.Lister()
cc.cmdSynced = cc.cmdInformer.Informer().HasSynced

cc.sharedInformers = informers.NewSharedInformerFactory(cc.kubeClients, 0)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

definitely this code is more clean

go cc.cmdInformer.Informer().Run(stopCh)
go cc.sharedInformers.Start(stopCh)
go cc.pcInformer.Informer().Run(stopCh)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The old cc.sharedInformers.Start(stopCh) is same as these lines.

@k82cn
Copy link
Member

k82cn commented Apr 29, 2019

let's only do code sync up in this PR; we can open other PRs to address comments.

@TommyLike
Copy link
Contributor Author

@k82cn There are some fundamental issues in master branch of CodeClud repo which lead the half failing of our testcases, for example, the pending state does not exist anymore, needs a fix before sync.

@k82cn
Copy link
Member

k82cn commented Apr 29, 2019

let's make sure code sync up firstly. We can fix the issues in different PRs.

k82cn added 3 commits April 29, 2019 20:53
Signed-off-by: Da K. Ma <klaus1982.cn@gmail.com>
Updated kube-batch to vk-kube-batch.
@TommyLike TommyLike changed the title [WIP] Sync Codes [WIP] Sync unrebased codes Apr 30, 2019
@TommyLike TommyLike changed the title [WIP] Sync unrebased codes Sync unrebased codes Apr 30, 2019
@k82cn k82cn closed this Apr 30, 2019
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.

6 participants