fix: remove extra task (#854)

Both the otlp agent and telemetry client themselves spawn tasks, we
probably don't need another task to do the spawning.

Author: astuyve

bottlecap/src/bin/bottlecap/main.rs

@@ -1109,11 +1109,9 @@ async fn setup_telemetry_client(
         TelemetryListener::new(EXTENSION_HOST_IP, TELEMETRY_PORT, logs_agent_channel);
 
     let cancel_token = telemetry_listener.cancel_token();
-    tokio::spawn(async move {
-        if let Err(e) = telemetry_listener.start() {
-            error!("Error starting telemetry listener: {e:?}");
-        }
-    });
+    if let Err(e) = telemetry_listener.start() {
+        error!("Error starting telemetry listener: {e:?}");
+    }
 
     telemetry::subscribe(
         client,
@@ -1140,11 +1138,9 @@ fn start_otlp_agent(
 
     let agent = OtlpAgent::new(config.clone(), tags_provider, trace_processor, trace_tx);
     let cancel_token = agent.cancel_token();
-    tokio::spawn(async move {
-        if let Err(e) = agent.start() {
-            error!("Error starting OTLP agent: {e:?}");
-        }
-    });
+    if let Err(e) = agent.start() {
+        error!("Error starting OTLP agent: {e:?}");
+    }
 
     Some(cancel_token)
 }