fix podgroup phase

[wangyuqing4]

when i use job.yaml

apiVersion: batch.volcano.sh/v1alpha1                                                                                                                                       
kind: Job 
metadata:
  generateName: job-xxx
spec:
  minAvailable: 4
  schedulerName: volcano
  plugins:
    ssh: []
    env: []
    svc: []
  policies:
  - event: PodFailed
    action: RestartJob
  tasks:
  - replicas: 1
    name: "mpimaster"
    template:
      spec:
        imagePullSecrets:
          - name: default-secret
        containers:
        - command:
            - /bin/sh
            - -c
            - | 
              mkdir -p /var/run/sshd; /usr/sbin/sshd; sleep 10;
          image: openmpi-hello:1.0
          imagePullPolicy: IfNotPresent
          name: mpimaster
          ports:
          - containerPort: 2222
            name: mpijob-port
          workingDir: /home
          resources:
            limits:
              cpu: 3750m
        restartPolicy: OnFailure
  - replicas: 3
    name: mpiworker
    template:
      spec:
        imagePullSecrets:
          - name: default-secret
        containers:
        - command:
            - /bin/sh
            - -c
            - |
              mkdir -p /var/run/sshd; /usr/sbin/sshd; sleep 5;
          image: openmpi-hello:1.0
          imagePullPolicy: IfNotPresent
          name: mpiworker
          ports:
          - containerPort: 2222
            name: mpijob-port
          workingDir: /home
          resources:
            limits:
              cpu: 3750m
        restartPolicy: OnFailure

i use kubectl create -f job.yaml, but phase of podgroup make me confused.

    | PG phase |        Pod             |
    | -------- | ---------------------- |
    | Pending  | none                   |
    | Inqueue  | 4 Pending              |
    | Running  | 4 Running              |
    | Pending  | 1 Running  3 Compeletd |
   

so i fix, when calc allocated num, include succeeded pod.

[hzxuzhonghu]

/lgtm

[hzxuzhonghu]

maybe fix #457, will have a test

[TravisBuddy]

Hey @wangyuqing4,
Your changes look good to me!

View build log

TravisBuddy Request Identifier: 9e459510-0116-11ea-82f1-cf3ba55cf6d0

[k82cn]

/lgtm
/approve

[volcano-sh-bot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: k82cn, wangyuqing4

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• pkg/scheduler/OWNERS [k82cn]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment