review fixes.

added explicit ignoring for write func

Author: jayantjain93

cluster-autoscaler/cloudprovider/gce/mig_target_sizes_provider.go

@@ -56,9 +56,9 @@ func (c *cachingMigTargetSizesProvider) GetMigTargetSize(migRef GceRef) (int64,
 
 	newTargetSizes, err := c.fillInMigTargetSizeCache()
 
-	// if we still do not have value here return an error
 	size, found := newTargetSizes[migRef]
 	if err != nil || !found {
+		// fallback to querying for single mig
 		targetSize, err = c.gceClient.FetchMigTargetSize(migRef)
 		if err != nil {
 			return 0, err

cluster-autoscaler/utils/test/test_utils.go

@@ -225,7 +225,7 @@ func NewHttpServerMock() *HttpServerMock {
 	mux.HandleFunc("/",
 		func(w http.ResponseWriter, req *http.Request) {
 			result := httpServerMock.handle(req.URL.Path)
-			w.Write([]byte(result))
+			_, _ = w.Write([]byte(result))
 		})
 
 	server := httptest.NewServer(mux)
@@ -246,8 +246,7 @@ func NewHttpServerMockWithStatusCode() *HttpServerMock {
 		func(w http.ResponseWriter, req *http.Request) {
 			code, result := httpServerMock.handleWithStatusCode(req.URL.Path)
 			w.WriteHeader(code)
-			w.Write([]byte(result))
-			//w.Write([]byte("status: " + string(code) + "\n" + result))
+			_, _ = w.Write([]byte(result))
 		})
 
 	server := httptest.NewServer(mux)