fix: improve tracing provider validation logic in OpsTraceManager

api/core/ops/ops_trace_manager.py

@@ -291,10 +291,14 @@ class OpsTraceManager:
         :return:
         """
         # auth check
-        try:
+        if enabled == True:
-            provider_config_map[tracing_provider]
+            try:
-        except KeyError:
+                provider_config_map[tracing_provider]
-            raise ValueError(f"Invalid tracing provider: {tracing_provider}")
+            except KeyError:
+                raise ValueError(f"Invalid tracing provider: {tracing_provider}")
+        else:
+            if tracing_provider is not None:
+                raise ValueError(f"Invalid tracing provider: {tracing_provider}")
 
         app_config: Optional[App] = db.session.query(App).filter(App.id == app_id).first()
         if not app_config: