Skip to content

Commit 4610998

Browse files
authored
Merge pull request #15 from humingcheng/master
Adapt to paas-lager
2 parents 0475a67 + da09f9d commit 4610998

File tree

3 files changed

+4
-4
lines changed

3 files changed

+4
-4
lines changed

csemonitor.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ func (reporter *Reporter) Run() {
8484
if err != nil {
8585
//If the connection fails for the first time then print Warn Logs
8686
if IsMonitoringConnected {
87-
lager.Logger.Warnf(err, "Unable to connect to monitoring server")
87+
lager.Logger.Warnf("Unable to connect to monitoring server, err: %v", err)
8888
}
8989
IsMonitoringConnected = false
9090
} else {

register.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ func reportMetricsToCSEDashboard(r metrics.Registry) error {
2020

2121
monitorServerURL, err := getMonitorEndpoint()
2222
if err != nil {
23-
lager.Logger.Warn("Get Monitoring URL failed, CSE monitoring function disabled", err)
23+
lager.Logger.Warnf("Get Monitoring URL failed, CSE monitoring function disabled, err: %v", err)
2424
return nil
2525
}
2626

utils.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ func getTLSForClient(monitorURL string) (*tls.Config, error) {
4141
}
4242
return nil, err
4343
}
44-
lager.Logger.Warnf(nil, "%s TLS mode, verify peer: %t, cipher plugin: %s",
44+
lager.Logger.Warnf("%s TLS mode, verify peer: %t, cipher plugin: %s",
4545
sslTag, sslConfig.VerifyPeer, sslConfig.CipherPlugin)
4646

4747
return tlsConfig, nil
@@ -70,7 +70,7 @@ func getMonitorEndpoint() (string, error) {
7070
if monitorEndpoint == "" {
7171
monitorURL, err := endpoint.GetEndpointFromServiceCenter("default", "CseMonitoring", "latest")
7272
if err != nil {
73-
lager.Logger.Warn("empty monitor server endpoint, please provide the monitor server endpoint", err)
73+
lager.Logger.Warnf("empty monitor server endpoint, please provide the monitor server endpoint, err: %v", err)
7474
return "", err
7575
}
7676

0 commit comments

Comments
 (0)