Correct telemetry log use (#12)

* Correct telemetry log use

* Add supported node types to README.md

README.md

@@ -7,6 +7,8 @@ run a Voi node.
 
 - Relay node
 - Archiver node
+- Developer node
+- Participation node
 
 Node type will default to 'relay' if not specified.
 Node configuration will use testnet defaults unless otherwise provided at image build time.

tools/start-node/start-node.go

@@ -84,6 +84,8 @@ func main() {
 			if err != nil {
 				log.Fatalf("Failed to set telemetry state: %v", err)
 			}
+
+			log.Printf("Telemetry %s. Telemetry Name: %s", map[bool]string{true: "enabled", false: "disabled"}[telemetrySet], telemetryName)
 		}
 
 		done = pu.StartProcess(algodCmd, "-d", algodDataDir)

tools/utils/file_utils.go

@@ -132,22 +132,18 @@ func (fu FileUtils) EnsureGUIDExists(filePath string) error {
 func (fu FileUtils) SetTelemetryState(filePath, telemetryName string, enabled bool) error {
 	err := fu.UpdateJSONAttribute(filePath, "Enable", enabled)
 	if err != nil {
-		log.Printf("failed to set telemetry enabled state: %v", err)
-		return err
+		return fmt.Errorf("failed to set telemetry enabled state: %v", err)
 	}
 	err = fu.UpdateJSONAttribute(filePath, "Name", telemetryName)
 	if err != nil {
-		log.Printf("failed to set telemetry name: %v", err)
-		return err
+		return fmt.Errorf("failed to set telemetry name: %v", err)
 	}
 
 	err = fu.EnsureGUIDExists(filePath)
 	if err != nil {
-		log.Printf("failed to ensure GUID exists: %v", err)
-		return err
+		return fmt.Errorf("failed to ensure GUID exists: %v", err)
 	}
 
-	log.Printf("Telemetry %s. Telemetry Name: %s", map[bool]string{true: "enabled", false: "disabled"}[enabled], telemetryName)
 	return nil
 }