Merge pull request #69087 from tanshanshan/error926

make sure to check for errors and close the request body in the cluster size autoscaling e2e test

Merge pull request #69087 from tanshanshan/error926
74f779fb · Kubernetes Prow Robot · GitHub · 173846b0 · 65af391e · 74f779fb
Unverified Commit 74f779fb authored Dec 29, 2018 by Kubernetes Prow Robot Committed by GitHub Dec 29, 2018
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 0 deletions

cluster_size_autoscaling.go test/e2e/autoscaling/cluster_size_autoscaling.go +5 -0

No files found.
--- a/test/e2e/autoscaling/cluster_size_autoscaling.go
+++ b/test/e2e/autoscaling/cluster_size_autoscaling.go
@@ -1195,6 +1195,11 @@ func executeHTTPRequest(method string, url string, body string) (string, error) 
 		return "", err
 	}
 	resp, err := client.Do(req)
+	if err != nil {
+		return "", err
+	}
+	defer resp.Body.Close()
+
 	respBody, err := ioutil.ReadAll(resp.Body)
 	if err != nil {
 		return "", err