Merge pull request #9 from killwing/dev-v1.9.7

add travis go test

Author: houz

.travis.yml

@@ -3,3 +3,7 @@ go:
     - "1.11"
 install:
     - mv $GOPATH/src/github.com/qiniu-ava $GOPATH/src/github.com/Microsoft
+script:
+    - make
+    - go test ./...
+

kube-scheduler/pkg/algorithm/predicates/predicates.go

@@ -513,7 +513,7 @@ func (c *VolumeZoneChecker) predicate(pod *v1.Pod, meta algorithm.PredicateMetad
 						class, _ := c.classInfo.GetStorageClassInfo(*scName)
 						if class != nil {
 							if class.VolumeBindingMode == nil {
-								return false, nil, fmt.Errorf("VolumeBindingMode not set for StorageClass %q", scName)
+								return false, nil, fmt.Errorf("VolumeBindingMode not set for StorageClass %q", *scName)
 							}
 							if *class.VolumeBindingMode == storagev1.VolumeBindingWaitForFirstConsumer {
 								// Skip unbound volumes

kube-scheduler/pkg/scheduler.go

@@ -331,7 +331,7 @@ func (sched *Scheduler) bindVolumesWorker() {
 		// The Pod is always sent back to the scheduler afterwards.
 		err := sched.config.VolumeBinder.Binder.BindPodVolumes(assumed)
 		if err != nil {
-			glog.V(1).Infof("Failed to bind volumes for pod \"%v/%v\"", assumed.Namespace, assumed.Name, err)
+			glog.V(1).Infof("Failed to bind volumes for pod \"%v/%v: %s\"", assumed.Namespace, assumed.Name, err)
 			reason = "VolumeBindingFailed"
 			eventType = v1.EventTypeWarning
 		} else {

kubeinterface/kubeinterface_test.go

@@ -62,6 +62,7 @@ func comparePod(pod0 *types.PodInfo, pod1 *types.PodInfo) {
 	if pod0.NodeName != pod1.NodeName {
 		fmt.Printf("Nodename does not match %s %s\n", pod0.NodeName, pod1.NodeName)
 	}
+
 	compareContainers(pod0.InitContainers, pod1.InitContainers)
 	compareContainers(pod0.RunningContainers, pod1.RunningContainers)
 }
@@ -124,7 +125,7 @@ func TestConvert(t *testing.T) {
 		ObjectMeta: metav1.ObjectMeta{
 			Name: "Pod0",
 			Annotations: map[string]string{
-				"ABCD": "EFGH",
+				"ABCD":                        "EFGH",
 				"pod.alpha/DeviceInformation": string(jsonStr),
 				// "PodInfo/InitContainer/Init0/Requests/alpha/grpresource/gpu/0/cards": "1",
 				// "PodInfo/InitContainer/Init0/Requests/alpha/grpresource/gpu/0/memory": "100000",
@@ -174,6 +175,7 @@ func TestConvert(t *testing.T) {
 	expectedPodInfo := &types.PodInfo{
 		Name:     "Pod0",
 		NodeName: "",
+		Requests: types.ResourceList{},
 		InitContainers: map[string]types.ContainerInfo{
 			"Init0": {
 				KubeRequests: types.ResourceList{"CPU": 4, "Memory": 100000, "Other": 20},
@@ -234,7 +236,7 @@ func TestConvert(t *testing.T) {
 
 	jsonStr, _ = json.Marshal(podInfo)
 	expectedAnnotations := map[string]string{
-		"ABCD": "EFGH", // existing
+		"ABCD":                        "EFGH", // existing
 		"pod.alpha/DeviceInformation": string(jsonStr),
 		// "PodInfo/InitContainer/Init0/Requests/alpha/grpresource/gpu/0/cards": "1",
 		// "PodInfo/InitContainer/Init0/Requests/alpha/grpresource/gpu/0/memory": "100000",