Failed to generate code suggestions for PR

Greptile Overview

Greptile Summary

Migrated service graph configuration from main config module to enterprise config module. The ServiceGraph struct (33 lines) was removed from src/config/src/config.rs, and all references were updated to use o2_enterprise::enterprise::common::config::get_config().service_graph instead of cfg.service_graph.

• Removed ServiceGraph struct and field from main Config in config.rs
• Updated service graph initialization check in main.rs
• Updated trace ingestion service graph check in service/traces/mod.rs
• Updated API stats endpoint to use enterprise config in service/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