Browse Source

fix create_tracing_app_config error (#19884)

tags/1.4.1
heyszt 5 months ago
parent
commit
11146b6bae
No account linked to committer's email address
2 changed files with 9 additions and 3 deletions
  1. 3
    1
      api/core/ops/ops_trace_manager.py
  2. 6
    2
      api/services/ops_service.py

+ 3
- 1
api/core/ops/ops_trace_manager.py View File

@@ -287,7 +287,9 @@ class OpsTraceManager:
:return:
"""
# auth check
if tracing_provider not in provider_config_map and tracing_provider is not None:
try:
provider_config_map[tracing_provider]
except KeyError:
raise ValueError(f"Invalid tracing provider: {tracing_provider}")

app_config: Optional[App] = db.session.query(App).filter(App.id == app_id).first()

+ 6
- 2
api/services/ops_service.py View File

@@ -87,7 +87,9 @@ class OpsService:
:param tracing_config: tracing config
:return:
"""
if tracing_provider not in provider_config_map and tracing_provider:
try:
provider_config_map[tracing_provider]
except KeyError:
return {"error": f"Invalid tracing provider: {tracing_provider}"}

config_class, other_keys = (
@@ -150,7 +152,9 @@ class OpsService:
:param tracing_config: tracing config
:return:
"""
if tracing_provider not in provider_config_map:
try:
provider_config_map[tracing_provider]
except KeyError:
raise ValueError(f"Invalid tracing provider: {tracing_provider}")

# check if trace config already exists

Loading…
Cancel
Save