UPSTREAM: <carry>: conditionally fill the UserAgent from the currently running test

OpenShift uses these function before any test is run and they cause NPE

OpenShift-Rebase-Source: 834af76

Author: p0lyn0mial

test/e2e/framework/util.go

@@ -518,9 +518,9 @@ func restclientConfig(kubeContext string) (*clientcmdapi.Config, error) {
 type ClientConfigGetter func() (*restclient.Config, error)
 
 // LoadConfig returns a config for a rest client with the UserAgent set to include the current test name.
-func LoadConfig() (config *restclient.Config, err error) {
+func LoadConfig(noUserAgent ...bool) (config *restclient.Config, err error) {
 	defer func() {
-		if err == nil && config != nil {
+		if err == nil && config != nil && len(noUserAgent) == 0 {
 			testDesc := ginkgo.CurrentSpecReport()
 			if len(testDesc.ContainerHierarchyTexts) > 0 {
 				testName := strings.Join(testDesc.ContainerHierarchyTexts, " ")
@@ -562,8 +562,8 @@ func LoadConfig() (config *restclient.Config, err error) {
 }
 
 // LoadClientset returns clientset for connecting to kubernetes clusters.
-func LoadClientset() (*clientset.Clientset, error) {
-	config, err := LoadConfig()
+func LoadClientset(noUserAgent ...bool) (*clientset.Clientset, error) {
+	config, err := LoadConfig(noUserAgent...)
 	if err != nil {
 		return nil, fmt.Errorf("error creating client: %v", err.Error())
 	}