fix: improve tracing provider validation logic in OpsTraceManager (#20042)

api/core/ops/ops_trace_manager.py

@@ -292,11 +292,14 @@ class OpsTraceManager:
         :return:
         """
         # auth check
-        if tracing_provider is not None:
+        if enabled == True:
             try:
                 provider_config_map[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: