fix BestEffort for overused

[chenyangxueHDU]

quick fix ths issue: All jobs are pending, when some jobs set resources，others not set resources.

[chenyangxueHDU]

/assign @k82cn

[TravisBuddy]

Travis tests have failed

Hey @chenyangxueHDU,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.

1st Build

View build log

make verify

go build -o _output/bin/deepcopy-gen ./cmd/deepcopy-gen/
_output/bin/deepcopy-gen -i ./pkg/apis/scheduling/v1alpha1/ -O zz_generated.deepcopy
hack/verify-gofmt.sh
diff -u ./test/e2e/e2e.go.orig ./test/e2e/e2e.go
--- ./test/e2e/e2e.go.orig	2018-10-11 13:22:31.547581760 +0000
+++ ./test/e2e/e2e.go	2018-10-11 13:22:31.547581760 +0000
@@ -221,7 +221,7 @@
 		err = waitTasksNotReady(context, job.Name, nn)
 		Expect(err).NotTo(HaveOccurred())
 	})
-	
+
 	It("Schedule BestEffort Job", func() {
 		context := initTestContext()
 		defer cleanupTestContext(context)
diff -u ./test/e2e/util.go.orig ./test/e2e/util.go
--- ./test/e2e/util.go.orig	2018-10-11 13:22:31.555578130 +0000
+++ ./test/e2e/util.go	2018-10-11 13:22:31.555578130 +0000
@@ -721,4 +721,3 @@
 		return taskNum <= readyTaskNum, nil
 	}
 }
-
make: *** [verify] Error 1

TravisBuddy Request Identifier: 7e440da0-cd5a-11e8-9005-69b5ac3c7904

[k82cn]

run ./hack/update-gofmt.sh to correct build error :)

[TravisBuddy]

Travis tests have failed

Hey @chenyangxueHDU,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.

1st Build

View build log

make verify

go build -o _output/bin/deepcopy-gen ./cmd/deepcopy-gen/
_output/bin/deepcopy-gen -i ./pkg/apis/scheduling/v1alpha1/ -O zz_generated.deepcopy
hack/verify-gofmt.sh
diff -u ./test/e2e/e2e.go.orig ./test/e2e/e2e.go
--- ./test/e2e/e2e.go.orig	2018-10-12 04:09:25.303854779 +0000
+++ ./test/e2e/e2e.go	2018-10-12 04:09:25.303854779 +0000
@@ -221,7 +221,7 @@
 		err = waitTasksNotReady(context, job.Name, nn)
 		Expect(err).NotTo(HaveOccurred())
 	})
-	
+
 	It("Schedule BestEffort Job", func() {
 		context := initTestContext()
 		defer cleanupTestContext(context)
make: *** [verify] Error 1

TravisBuddy Request Identifier: 6f530f90-cdd6-11e8-9005-69b5ac3c7904

[k82cn]

/approve

please combine those commits into one; I'll lgtm when CI is happy.

BTW, please rebase for #419

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

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

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:

• OWNERS [k82cn]

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

[chenyangxueHDU]

/approve

please combine those commits into one; I'll lgtm when CI is happy.

BTW, please rebase for #419

Done :)

[TravisBuddy]

Travis tests have failed

Hey @chenyangxueHDU,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.

1st Build

View build log

make verify

go build -o _output/bin/deepcopy-gen ./cmd/deepcopy-gen/
_output/bin/deepcopy-gen -i ./pkg/apis/scheduling/v1alpha1/ -O zz_generated.deepcopy
hack/verify-gofmt.sh
diff -u ./test/e2e/e2e.go.orig ./test/e2e/e2e.go
--- ./test/e2e/e2e.go.orig	2018-10-12 10:10:01.706163839 +0000
+++ ./test/e2e/e2e.go	2018-10-12 10:10:01.706163839 +0000
@@ -221,7 +221,7 @@
 		err = waitTasksNotReady(context, job.Name, nn)
 		Expect(err).NotTo(HaveOccurred())
 	})
-	
+
 	It("Schedule BestEffort Job", func() {
 		context := initTestContext()
 		defer cleanupTestContext(context)
make: *** [verify] Error 1

TravisBuddy Request Identifier: d05ac530-ce08-11e8-b4b8-efdbea9fb873

[k82cn]

/lgtm