scriptoriumadmin/litellm
1
0
Fork 0
mirror of https://github.com/BerriAI/litellm.git synced 2026-09-05 08:07:05 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 136

refactor: resolve review comments

Browse source
This commit is contained in:
Nagesh Bansal 2026-08-26 18:07:19 +05:30
parent 0d67802832
commit 2ddd7a7d67
No known key found for this signature in database
GPG key ID: 87AE46945352DBF2
1 changed files with 6 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
litellm/integrations/otel/presets/signoz.py
View file

@ -20,8 +20,7 @@ SIGNOZ_INGESTION_ENDPOINT_ENV: Final = "SIGNOZ_INGESTION_ENDPOINT"
class _SigNozSettings(BaseSettings):
model_config = SettingsConfigDict(case_sensitive=False, extra="ignore")
# One endpoint for both SigNoz Cloud and self-hosted, with no default host, so
# nothing exports until the operator names a destination
# No default host: nothing exports until the operator names a destination
endpoint: str | None = Field(default=None, validation_alias=SIGNOZ_INGESTION_ENDPOINT_ENV)
ingestion_key: str | None = Field(default=None, validation_alias="SIGNOZ_INGESTION_KEY")
@ -52,12 +51,11 @@ def signoz_preset(
def signoz_dynamic_endpoint(params: StandardCallbackDynamicParams) -> str | None:
"""Per-request SigNoz endpoint from team/key dynamic params.
``None`` keeps the operator's endpoint, so a team that saved only a key still
exports to the configured destination.
"""
return params.get("signoz_ingestion_endpoint")
"""Per-request SigNoz endpoint from team/key dynamic params; ``None`` keeps the operator's."""
endpoint: Final = params.get("signoz_ingestion_endpoint")
if not endpoint or not endpoint.startswith(("http://", "https://")):
return None
return endpoint
def signoz_dynamic_headers(