Add basic config file options to otel telemetry (#330)

* Add basic config file options to otel telemetry

* Happy path unit test for telemetry config

* Changeset

* Taplo format

* Refactor to add a couple more tests

* Update unit test for clippy after rust upgrade

* Rename unit tests

Author: swcollard

.changesets/config_telemetry_config.md

@@ -0,0 +1,7 @@
+### Add basic config file options to otel telemetry - @swcollard PR #330
+
+Adds new Configuration options for setting up configuration beyond the standard OTEL environment variables needed before.
+
+* Renames trace->telemetry
+* Adds OTLP options for metrics and tracing to choose grpc or http upload protocols and setting the endpoints
+* This configuration is all optional, so by default nothing will be logged

Cargo.lock

@@ -32,7 +32,13 @@ jwks = "0.4.0"
 lz-str = "0.2.1"
 opentelemetry = "0.30.0"
 opentelemetry-appender-log = "0.30.0"
-opentelemetry-otlp = "0.30.0"
+opentelemetry-otlp = { version = "0.30.0", features = [
+  "grpc-tonic",
+  "tonic",
+  "http-proto",
+  "metrics",
+  "trace",
+] }
 opentelemetry-resource-detectors = "0.9.0"
 opentelemetry-semantic-conventions = "0.30.0"
 opentelemetry-stdout = "0.30.0"

crates/apollo-mcp-server/Cargo.toml

@@ -449,27 +449,27 @@ mod test {
                 .find(|scope_metrics| scope_metrics.scope().name() == "apollo.mcp")
             {
                 for metric in scope_metrics.metrics() {
-                    if metric.name() == "apollo.mcp.operation.count" {
-                        if let AggregatedMetrics::U64(MetricData::Sum(data)) = metric.data() {
-                            for point in data.data_points() {
-                                let attributes = point.attributes();
-                                let mut attr_map = std::collections::HashMap::new();
-                                for kv in attributes {
-                                    attr_map.insert(kv.key.as_str(), kv.value.as_str());
-                                }
-                                assert_eq!(
-                                    attr_map.get("operation.id").map(|s| s.as_ref()),
-                                    Some("mock_operation")
-                                );
-                                assert_eq!(
-                                    attr_map.get("operation.type").map(|s| s.as_ref()),
-                                    Some("persisted_query")
-                                );
-                                assert_eq!(
-                                    attr_map.get("success"),
-                                    Some(&std::borrow::Cow::Borrowed("false"))
-                                );
+                    if metric.name() == "apollo.mcp.operation.count"
+                        && let AggregatedMetrics::U64(MetricData::Sum(data)) = metric.data()
+                    {
+                        for point in data.data_points() {
+                            let attributes = point.attributes();
+                            let mut attr_map = std::collections::HashMap::new();
+                            for kv in attributes {
+                                attr_map.insert(kv.key.as_str(), kv.value.as_str());
                             }
+                            assert_eq!(
+                                attr_map.get("operation.id").map(|s| s.as_ref()),
+                                Some("mock_operation")
+                            );
+                            assert_eq!(
+                                attr_map.get("operation.type").map(|s| s.as_ref()),
+                                Some("persisted_query")
+                            );
+                            assert_eq!(
+                                attr_map.get("success"),
+                                Some(&std::borrow::Cow::Borrowed("false"))
+                            );
                         }
                     }
                 }

crates/apollo-mcp-server/src/graphql.rs

@@ -41,7 +41,7 @@ async fn main() -> anyhow::Result<()> {
         None => runtime::read_config_from_env().unwrap_or_default(),
     };
 
-    let _guard = runtime::trace::init_tracing_subscriber(&config)?;
+    let _guard = runtime::telemetry::init_tracing_subscriber(&config)?;
 
     info!(
         "Apollo MCP Server v{} // (c) Apollo Graph, Inc. // Licensed under MIT",

crates/apollo-mcp-server/src/main.rs

@@ -12,7 +12,7 @@ mod operation_source;
 mod overrides;
 mod schema_source;
 mod schemas;
-pub mod trace;
+pub mod telemetry;
 
 use std::path::Path;
 
@@ -270,6 +270,11 @@ mod test {
                     path: None,
                     rotation: Hourly,
                 },
+                telemetry: Telemetry {
+                    exporters: None,
+                    service_name: None,
+                    version: None,
+                },
                 operations: Infer,
                 overrides: Overrides {
                     disable_type_description: false,

crates/apollo-mcp-server/src/runtime.rs

@@ -8,7 +8,7 @@ use url::Url;
 
 use super::{
     OperationSource, SchemaSource, endpoint::Endpoint, graphos::GraphOSConfig,
-    introspection::Introspection, logging::Logging, overrides::Overrides,
+    introspection::Introspection, logging::Logging, overrides::Overrides, telemetry::Telemetry,
 };
 
 /// Configuration for the MCP server
@@ -43,6 +43,9 @@ pub struct Config {
     /// Logging configuration
     pub logging: Logging,
 
+    /// Telemetry configuration
+    pub telemetry: Telemetry,
+
     /// Operations
     pub operations: OperationSource,