Merge pull request #67188 from aleksandra-malinowska/autoscaling-test-fix-21

Automatic merge from submit-queue (batch tested with PRs 66507, 67188). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. Log error in autoscaling e2e tests when creating priority classes Log error in autoscaling e2e tests when creating priority classes ```release-note NONE ``` /cc @MaciekPytel /sig autoscaling

Merge pull request #67188 from aleksandra-malinowska/autoscaling-test-fix-21
91f93b74 · Kubernetes Submit Queue · GitHub · 8ebc84e7 · 73790b8f · 91f93b74
Unverified Commit 91f93b74 authored Aug 09, 2018 by Kubernetes Submit Queue Committed by GitHub Aug 09, 2018
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 1 deletion

cluster_size_autoscaling.go test/e2e/autoscaling/cluster_size_autoscaling.go +7 -1

No files found.
--- a/test/e2e/autoscaling/cluster_size_autoscaling.go
+++ b/test/e2e/autoscaling/cluster_size_autoscaling.go
@@ -1926,12 +1926,18 @@ func createPriorityClasses(f *framework.Framework) func() {
 	}
 	for className, priority := range priorityClasses {
 		_, err := f.ClientSet.SchedulingV1beta1().PriorityClasses().Create(&schedulerapi.PriorityClass{ObjectMeta: metav1.ObjectMeta{Name: className}, Value: priority})
+		if err != nil {
+			glog.Errorf("Error creating priority class: %v", err)
+		}
 		Expect(err == nil || errors.IsAlreadyExists(err)).To(Equal(true))
 	}
 	return func() {
 		for className := range priorityClasses {
-			f.ClientSet.SchedulingV1beta1().PriorityClasses().Delete(className, nil)
+			err := f.ClientSet.SchedulingV1beta1().PriorityClasses().Delete(className, nil)
+			if err != nil {
+				glog.Errorf("Error deleting priority class: %v", err)
+			}
 		}
 	}
 }