github/kubernetes
4
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Improved logging for horizontal pod autoscaler.

Browse Source 
Improved logging for horizontal pod autoscaler. Disabled redundant e2e test.
This commit is contained in:
Jerzy Szczepkowski 2015-10-05 16:26:14 +02:00
parent b9cfab87e3
commit f8b18b0a4c
2 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
pkg/controller/podautoscaler/horizontal.go
View File

@ -131,6 +131,8 @@ func (a *HorizontalController) reconcileAutoscaler(hpa experimental.HorizontalPo
return fmt.Errorf("failed to rescale %s: %v", reference, err) return fmt.Errorf("failed to rescale %s: %v", reference, err)
} }
a.eventRecorder.Eventf(&hpa, "SuccessfulRescale", "New size: %d", desiredReplicas) a.eventRecorder.Eventf(&hpa, "SuccessfulRescale", "New size: %d", desiredReplicas)
glog.Infof("Successfull rescale of %s, old size: %d, new size: %d, usage ratio: %f",
hpa.Name, currentReplicas, desiredReplicas, usageRatio)
} else { } else {
desiredReplicas = currentReplicas desiredReplicas = currentReplicas
} }

2
test/e2e/horizontal_pod_autoscaling.go
View File

@ -72,7 +72,7 @@ var _ = Describe("Horizontal pod autoscaling", func() {
}) })
// Backup tests, currently disabled // Backup tests, currently disabled
It("[Skipped][Autoscaling Suite] should scale from 1 pod to 3 pods (scale resource: CPU)", func() { It("[Skipped][Horizontal pod autoscaling Suite] should scale from 1 pod to 3 pods (scale resource: CPU)", func() {
rc = NewDynamicResourceConsumer("rc", 1, 700, 0, 800, 100, f) rc = NewDynamicResourceConsumer("rc", 1, 700, 0, 800, 100, f)
defer rc.CleanUp() defer rc.CleanUp()
createCPUHorizontalPodAutoscaler(rc, "0.3") createCPUHorizontalPodAutoscaler(rc, "0.3")