Skip to content
This repository has been archived by the owner on May 6, 2022. It is now read-only.

fix invalid log #2429

Merged
merged 2 commits into from
Oct 23, 2018
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 5 additions & 4 deletions pkg/controller/broker_client_manager.go
Original file line number Diff line number Diff line change
Expand Up @@ -18,10 +18,11 @@ package controller

import (
"fmt"
"github.com/golang/glog"
osb "github.com/pmorie/go-open-service-broker-client/v2"
"reflect"
"sync"

"github.com/golang/glog"
osb "github.com/pmorie/go-open-service-broker-client/v2"
carlory marked this conversation as resolved.
Show resolved Hide resolved
)

// BrokerKey defines a key which points to a broker (cluster wide or namespaced)
Expand Down Expand Up @@ -84,7 +85,7 @@ func (m *BrokerClientManager) UpdateBrokerClient(brokerKey BrokerKey, clientConf
existing, found := m.clients[brokerKey]

if !found || configHasChanged(existing.clientConfig, clientConfig) {
glog.V(4).Infof("Updating OSB client for broker %s, URL: %s", brokerKey.String(), clientConfig.URL)
glog.V(4).Infof("Updating OSB client for broker %q, URL: %s", brokerKey.String(), clientConfig.URL)
return m.createClient(brokerKey, clientConfig)
}

Expand All @@ -96,7 +97,7 @@ func (m *BrokerClientManager) RemoveBrokerClient(brokerKey BrokerKey) {
m.mu.Lock()
defer m.mu.Unlock()

glog.V(4).Info("Removing OSB client for broker %s", brokerKey.String())
glog.V(4).Infof("Removing OSB client for broker %q", brokerKey.String())
delete(m.clients, brokerKey)
}

Expand Down