Merge pull request #30559 from mwielgus/informer-quick-fix

Automatic merge from submit-queue

Fix in default client factory in federated informer

This wasn't catched by junit tests because we mock the factory.
This commit is contained in:
Kubernetes Submit Queue 2016-08-12 18:38:48 -07:00 committed by GitHub
commit b7ccc15b0a

View File

@ -122,7 +122,7 @@ func NewFederatedInformer(
targetInformerFactory: targetInformerFactory,
clientFactory: func(cluster *federation_api.Cluster) (federation_release_1_4.Interface, error) {
clusterConfig, err := BuildClusterConfig(cluster)
if err != nil && clusterConfig != nil {
if err == nil && clusterConfig != nil {
clientset := federation_release_1_4.NewForConfigOrDie(restclient.AddUserAgent(clusterConfig, userAgentName))
return clientset, nil
}