functions called Run should not return

node/node.go

@@ -144,10 +144,10 @@ func (w *ConfigSyncer) updateConfigWithResponse(key, value string) {
 	}
 }
 
-// Run the ConfigSyncer in the background. It will wait for
+// Start the ConfigSyncer in the background. It will wait for
 // initialization to complete, so that when this function returns, the
 // in-memory configuration has already been fully synchronized.
-func (w *ConfigSyncer) Run() {
+func (w *ConfigSyncer) Start() {
 	// Run until the first successful Get().
 	log.Printf("attempting to retrieve initial config...")
 	for {
@@ -280,7 +280,7 @@ func (rc *RadioNode) Run() {
 	// Bootstrap the config watcher. This ensures that we have a
 	// full configuration (thanks to the Get() call) before we
 	// start managing the icecast server.
-	rc.watcher.Run()
+	rc.watcher.Start()
 
 	// Start the presence heartbeat.
 	go rc.presence()