Skip to content

Commit 2eb1ccd

Browse files
passed as reference instead of value (Azure#483)
1 parent a151c83 commit 2eb1ccd

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

cns/configuration/configuration.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ func ReadConfig() (CNSConfig, error) {
6969
}
7070

7171
// set telmetry setting defaults
72-
func setTelemetrySettingDefaults(telemetrySettings TelemetrySettings) {
72+
func setTelemetrySettingDefaults(telemetrySettings *TelemetrySettings) {
7373
if telemetrySettings.RefreshIntervalInSecs == 0 {
7474
// set the default refresh interval of metadata thread to 15 seconds
7575
telemetrySettings.RefreshIntervalInSecs = 15
@@ -93,5 +93,5 @@ func setTelemetrySettingDefaults(telemetrySettings TelemetrySettings) {
9393

9494
// Set Default values of CNS config if not specified
9595
func SetCNSConfigDefaults(config *CNSConfig) {
96-
setTelemetrySettingDefaults(config.TelemetrySettings)
96+
setTelemetrySettingDefaults(&config.TelemetrySettings)
9797
}

cns/service/main.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -224,8 +224,8 @@ func main() {
224224
logger.Errorf("[Azure CNS] Error reading cns config: %v", err)
225225
}
226226

227-
logger.Printf("[Azure CNS] Read config :%+v", cnsconfig)
228227
configuration.SetCNSConfigDefaults(&cnsconfig)
228+
logger.Printf("[Azure CNS] Read config :%+v", cnsconfig)
229229

230230
disableTelemetry := cnsconfig.TelemetrySettings.DisableAll
231231

0 commit comments

Comments
 (0)