Failed to generate code suggestions for PR

Greptile Overview

Greptile Summary

Migrated service graph configuration from main config module to enterprise config module. TheServiceGraph struct (33 lines) was removed fromsrc/config/src/config.rs, and all references were updated to useo2_enterprise::enterprise::common::config::get_config().service_graph instead ofcfg.service_graph.

• RemovedServiceGraph struct and field from mainConfig inconfig.rs
• Updated service graph initialization check inmain.rs
• Updated trace ingestion service graph check inservice/traces/mod.rs
• Updated API stats endpoint to use enterprise config inservice/traces/service_graph/api.rs

This change correctly consolidates enterprise-feature configuration under the enterprise module, improving code organization and maintaining feature flag separation.

Confidence Score: 5/5

• This PR is safe to merge - straightforward refactoring with consistent patterns
• Clean config migration with no logic changes - all service graph config references consistently moved from main config to enterprise config module, following the existing pattern used by other enterprise features
• No files require special attention

Important Files Changed

File Analysis

Sequence Diagram

sequenceDiagram    participant App as Application    participant MainConfig as config::get_config()    participant EntConfig as o2_enterprise::...::get_config()    participant ServiceGraph as Service Graph Module    Note over App,ServiceGraph: Before PR (Old Pattern)    App->>MainConfig: Check cfg.service_graph.enabled    MainConfig-->>App: Returns bool    App->>ServiceGraph: Initialize if enabled    Note over App,ServiceGraph: After PR (New Pattern)    App->>EntConfig: Check .service_graph.enabled    EntConfig-->>App: Returns bool    App->>ServiceGraph: Initialize if enabled        Note over MainConfig,EntConfig: Config Migration    Note over MainConfig: ServiceGraph struct removed    Note over EntConfig: ServiceGraph config now here