Merge pull request #37364 from foxyriver/status-code

Automatic merge from submit-queue use status code const to express http status  **What this PR does / why we need it**: Using status code const to express http status. **Special notes for your reviewer**: **Release note**:  ```release-note ```

Merge pull request #37364 from foxyriver/status-code
910912ea · Kubernetes Submit Queue · GitHub · 058c345b · ee6568a5 · 910912ea
Commit 910912ea authored Dec 07, 2016 by Kubernetes Submit Queue Committed by GitHub Dec 07, 2016
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

visitor.go pkg/kubectl/resource/visitor.go +1 -1

http.go pkg/kubelet/config/http.go +1 -1

No files found.
--- a/pkg/kubectl/resource/visitor.go
+++ b/pkg/kubectl/resource/visitor.go
@@ -263,7 +263,7 @@ func readHttpWithRetries(get httpget, duration time.Duration, u string, attempts
 		}

 		// Error - Set the error condition from the StatusCode
-		if statusCode != 200 {
+		if statusCode != http.StatusOK {
 			err = fmt.Errorf("unable to read URL %q, server reported %s, status code=%d", u, status, statusCode)
 		}


--- a/pkg/kubelet/config/http.go
+++ b/pkg/kubelet/config/http.go
@@ -97,7 +97,7 @@ func (s *sourceURL) extractFromURL() error {
 	if err != nil {
 		return err
 	}
-	if resp.StatusCode != 200 {
+	if resp.StatusCode != http.StatusOK {
 		return fmt.Errorf("%v: %v", s.url, resp.Status)
 	}
 	if len(data) == 0 {