Avoid to create an API client if not needed

Author: fabianofranz

Diff for: pkg/cmd/util/clientcmd/factory.go

@@ -328,17 +328,16 @@ func NewFactory(clientConfig kclientcmd.ClientConfig) *Factory {
 	}
 	kLogsForObjectFunc := w.Factory.LogsForObject
 	w.LogsForObject = func(object, options runtime.Object) (*restclient.Request, error) {
-		oc, _, err := w.Clients()
-		if err != nil {
-			return nil, err
-		}
-
 		switch t := object.(type) {
 		case *deployapi.DeploymentConfig:
 			dopts, ok := options.(*deployapi.DeploymentLogOptions)
 			if !ok {
 				return nil, errors.New("provided options object is not a DeploymentLogOptions")
 			}
+			oc, _, err := w.Clients()
+			if err != nil {
+				return nil, err
+			}
 			return oc.DeploymentLogs(t.Namespace).Get(t.Name, *dopts), nil
 		case *buildapi.Build:
 			bopts, ok := options.(*buildapi.BuildLogOptions)
@@ -348,12 +347,20 @@ func NewFactory(clientConfig kclientcmd.ClientConfig) *Factory {
 			if bopts.Version != nil {
 				return nil, errors.New("cannot specify a version and a build")
 			}
+			oc, _, err := w.Clients()
+			if err != nil {
+				return nil, err
+			}
 			return oc.BuildLogs(t.Namespace).Get(t.Name, *bopts), nil
 		case *buildapi.BuildConfig:
 			bopts, ok := options.(*buildapi.BuildLogOptions)
 			if !ok {
 				return nil, errors.New("provided options object is not a BuildLogOptions")
 			}
+			oc, _, err := w.Clients()
+			if err != nil {
+				return nil, err
+			}
 			builds, err := oc.Builds(t.Namespace).List(api.ListOptions{})
 			if err != nil {
 				return nil, err