fix(query): Some environment variable configurations do not work (#18384)

* x

* fix

---------

Co-authored-by: sundyli <[email protected]>

Author: forsaken628

Cargo.lock

@@ -488,7 +488,7 @@ serde_test = "1.0"
 serde_urlencoded = "0.7.1"
 serde_with = { version = "3.8.1" }
 serde_yaml = { version = "0.9.34" }
-serfig = "0.1.0"
+serfig = { git = "https://github.com/datafuse-extras/serfig", rev = "610ac6d" }
 sha1 = "0.10.5"
 sha2 = "0.10.8"
 simdutf8 = "0.1.4"

Cargo.toml

@@ -3529,7 +3529,7 @@ pub struct SpillConfig {
     /// Allow space in bytes to spill to local disk.
     pub spill_local_disk_max_bytes: u64,
 
-    // TODO: We need to fix StorageConfig so that it supports environment variables and command line injections.
+    // TODO: We need to fix StorageConfig so that it supports command line injections.
     #[clap(skip)]
     pub storage: Option<StorageConfig>,
 }
@@ -3840,4 +3840,19 @@ mod test {
             "default setting is different from default config, please check again"
         )
     }
+
+    #[test]
+    fn test_env() {
+        unsafe {
+            std::env::set_var("LOG_TRACING_OTLP_ENDPOINT", "http://127.0.2.1:1111");
+            std::env::set_var("LOG_TRACING_CAPTURE_LOG_LEVEL", "DebuG");
+        }
+
+        let cfg = Config::load_with_config_file("").unwrap();
+        assert_eq!(
+            cfg.log.tracing.tracing_otlp.endpoint,
+            "http://127.0.2.1:1111"
+        );
+        assert_eq!(cfg.log.tracing.tracing_capture_log_level, "DebuG");
+    }
 }