should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...)

pkg/cloudprovider/providers/mesos/mesos.go

@@ -169,7 +169,7 @@ func (c *MesosCloud) Master(clusterName string) (string, error) {
 			return host, nil
 		}
 	}
-	return "", errors.New(fmt.Sprintf("The supplied cluster '%v' does not exist", clusterName))
+	return "", fmt.Errorf("The supplied cluster '%v' does not exist", clusterName)
 }
 
 // ipAddress returns an IP address of the specified instance.

pkg/volume/iscsi/iscsi_util.go

@@ -125,7 +125,7 @@ func (util *ISCSIUtil) AttachDisk(b iscsiDiskMounter) error {
 
 		if iscsiTransport == "" {
 			glog.Errorf("iscsi: could not find transport name in iface %s", b.iface)
-			return errors.New(fmt.Sprintf("Could not parse iface file for %s", b.iface))
+			return fmt.Errorf("Could not parse iface file for %s", b.iface)
 		} else if iscsiTransport == "tcp" {
 			devicePath = strings.Join([]string{"/dev/disk/by-path/ip", tp, "iscsi", b.iqn, "lun", b.lun}, "-")
 		} else {

test/e2e/kubectl.go

@@ -20,7 +20,6 @@ import (
 	"bytes"
 	"context"
 	"encoding/json"
-	"errors"
 	"fmt"
 	"io"
 	"io/ioutil"
@@ -1837,7 +1836,7 @@ func getUDData(jpgExpected string, ns string) func(clientset.Interface, string)
 		if strings.Contains(data.Image, jpgExpected) {
 			return nil
 		} else {
-			return errors.New(fmt.Sprintf("data served up in container is inaccurate, %s didn't contain %s", data, jpgExpected))
+			return fmt.Errorf("data served up in container is inaccurate, %s didn't contain %s", data, jpgExpected)
 		}
 	}
 }

test/e2e_node/environment/conformance.go

@@ -82,7 +82,7 @@ func check(options ...string) []error {
 			errs = appendNotNil(errs, kernel())
 		default:
 			fmt.Printf("Unrecognized option %s", c)
-			errs = append(errs, errors.New(fmt.Sprintf("Unrecognized option %s", c)))
+			errs = append(errs, fmt.Errorf("Unrecognized option %s", c))
 		}
 	}
 	return errs