Merge pull request #513 from wangyuqing4/cleanup

volcano-sh-bot · web-flow · commit 124d6aad38a8 · 2019-11-07T14:05:59.000+08:00
fix podgroup phase
diff --git a/pkg/scheduler/framework/session.go b/pkg/scheduler/framework/session.go
@@ -174,7 +174,7 @@ func jobStatus(ssn *Session, jobInfo *api.JobInfo) scheduling.PodGroupStatus {
 	} else {
 		allocated := 0
 		for status, tasks := range jobInfo.TaskStatusIndex {
-			if api.AllocatedStatus(status) {
+			if api.AllocatedStatus(status) || status == api.Succeeded {
 				allocated += len(tasks)
 			}
 		}

Original file line number	Diff line number	Diff line change
`@@ -174,7 +174,7 @@ func jobStatus(ssn Session, jobInfo api.JobInfo) scheduling.PodGroupStatus {`
`174`	`174`	`} else {`
`175`	`175`	`allocated := 0`
`176`	`176`	`for status, tasks := range jobInfo.TaskStatusIndex {`
`177`		`- if api.AllocatedStatus(status) {`
	`177`	`+ if api.AllocatedStatus(status) \|\| status == api.Succeeded {`
`178`	`178`	`allocated += len(tasks)`
`179`	`179`	`}`
`180`	`180`	`}`