Merge pull request #255 from lminzhw/job_status

init job status to Pending

pkg/apis/batch/v1alpha1/job.go

@@ -114,6 +114,8 @@ const (
 	PodGroupError JobEvent = "PodGroupError"
 	//ExecuteAction action issued event for each action
 	ExecuteAction JobEvent = "ExecuteAction"
+	//JobStatusError is generated if update job status failed
+	JobStatusError JobEvent = "JobStatusError"
 )
 
 // Event represent the phase of Job, e.g. pod-failed.

pkg/controllers/job/job_controller_actions.go

@@ -154,6 +154,12 @@ func (cc *Controller) createJob(jobInfo *apis.JobInfo, updateStatus state.Update
 	job := jobInfo.Job.DeepCopy()
 	glog.Infof("Current Version is: %d of job: %s/%s", job.Status.Version, job.Namespace, job.Name)
 
+	if err := cc.initJobStatus(job); err != nil {
+		cc.recorder.Event(job, v1.EventTypeWarning, string(vkv1.JobStatusError),
+			fmt.Sprintf("Failed to initialize job status, err: %v", err))
+		return err
+	}
+
 	if err := cc.pluginOnJobAdd(job); err != nil {
 		cc.recorder.Event(job, v1.EventTypeWarning, string(vkv1.PluginError),
 			fmt.Sprintf("Execute plugin when job add failed, err: %v", err))
@@ -523,3 +529,25 @@ func (cc *Controller) calcPGMinResources(job *vkv1.Job) *v1.ResourceList {
 
 	return &minAvailableTasksRes
 }
+
+func (cc *Controller) initJobStatus(job *vkv1.Job) error {
+	if job.Status.State.Phase != "" {
+		return nil
+	}
+
+	job.Status.State.Phase = vkv1.Pending
+	job.Status.MinAvailable = int32(job.Spec.MinAvailable)
+	job, err := cc.vkClients.BatchV1alpha1().Jobs(job.Namespace).UpdateStatus(job)
+	if err != nil {
+		glog.Errorf("Failed to update status of Job %v/%v: %v",
+			job.Namespace, job.Name, err)
+		return err
+	}
+	if err := cc.cache.Update(job); err != nil {
+		glog.Errorf("CreateJob - Failed to update Job %v/%v in cache:  %v",
+			job.Namespace, job.Name, err)
+		return err
+	}
+
+	return nil
+}