mirror of
https://github.com/BerriAI/litellm.git
synced 2026-08-28 05:25:59 +00:00
* test(e2e): harden stage flakes for batches, UI, and MCP Unique batch model names avoid load-balancing onto stale azure-batch deployments that still pointed at the retired gpt-4.1-mini-batch, which only the managed/unified path was hitting. Retry batch retrieve on 500 and /ui/api-keys navigation on ERR_ABORTED. Skip the MCP key-access suite when the compose-only mcp-upstream is unreachable on stage k8s * test(e2e): cover Datadog remote MCP via search_datadog_logs Register the regional Datadog MCP endpoint with DD-API-KEY / DD-APPLICATION-KEY static headers (CI-safe header auth; browser OAuth is not headless-automatable). Seed a chat completion marked e2e-datadog-mcp-*, assert the proxy shipped it, list tools, call search_datadog_logs for the marker, and delete the server on teardown. Math-upstream key-access tests only skip when that compose service is unreachable * test(e2e): drop compose math MCP upstream; use Datadog only Key-access denial and happy-path MCP e2e both register the real regional Datadog remote MCP server with DD-API-KEY / DD-APPLICATION-KEY headers. Remove the mcp-upstream compose service and FastMCP add/multiply fixture * docs(e2e): require real Datadog MCP for all mcp suite tests Document that tests/e2e/mcp must register via datadog_mcp helpers against mcp.<site>/v1/mcp and must not introduce compose or fake MCP upstreams * chore: restore mcp_e2e_upstream_server.py Keep the FastMCP fixture file; e2e no longer wires it in compose, but the module itself is not part of the Datadog-only cleanup * fix(e2e): load tests/e2e/.env and fix datadog_reader importlib load pytest on the host never inherited compose env_file keys, so DD_API_KEY stayed empty. load_dotenv tests/e2e/.env in e2e_config. Register the dynamically loaded datadog_reader module in sys.modules so dataclasses do not crash under Python 3.12 * test(e2e/batches): harden azure/vertex unified lifecycle flakes Put the provider deployment name in every JSONL body so Azure does not depend on a perfect model rewrite. Retry create/retrieve/cancel on transient statuses with backoff. Drop cancel assertions for azure and vertex (registry only has a shared basic cell; create+retrieve prove routing, cancel stays best-effort cleanup) * test(e2e/ui): treat api-keys shell as success after SPA ERR_ABORTED Post-login client redirects abort the first /ui/api-keys/ goto on stage. Wait off /ui/login after cookie set, then accept the page once Create New Key is visible even if goto raised ERR_ABORTED * test(e2e): drop flaky key models dropdown Playwright suite API management e2e already covers key generate/update persistence. The UI Models-dropdown sentinel cases only added SPA ERR_ABORTED noise and no unique product signal. Remove the suite and unused browser fixtures
258 lines
8.1 KiB
Python
258 lines
8.1 KiB
Python
"""Provider x routing-scenario matrix for the batches lifecycle e2e."""
|
|
|
|
from __future__ import annotations
|
|
|
|
import base64
|
|
import os
|
|
from dataclasses import dataclass
|
|
from typing import Literal
|
|
|
|
from e2e_config import unique_marker
|
|
from models import LiteLLMParamsBody
|
|
|
|
_BATCH_RUN = unique_marker()
|
|
|
|
|
|
def batch_model_name(base: str) -> str:
|
|
return f"{base}-{_BATCH_RUN}"
|
|
|
|
|
|
def _env_ref(*names: str) -> str:
|
|
for name in names:
|
|
value = os.environ.get(name)
|
|
if value is not None and value.strip() != "":
|
|
return f"os.environ/{name}"
|
|
return f"os.environ/{names[0]}"
|
|
|
|
Scenario = Literal["encoded", "unified", "model_param", "provider_fallback"]
|
|
|
|
IdShape = Literal["managed", "model_encoded", "raw"]
|
|
|
|
SCENARIOS: tuple[Scenario, ...] = (
|
|
"encoded",
|
|
"unified",
|
|
"model_param",
|
|
"provider_fallback",
|
|
)
|
|
|
|
|
|
@dataclass(frozen=True, slots=True)
|
|
class Provider:
|
|
name: str
|
|
model: str
|
|
raw_model: str
|
|
can_cancel: bool
|
|
can_list: bool
|
|
|
|
def litellm_params(self) -> LiteLLMParamsBody:
|
|
match self.name:
|
|
case "openai":
|
|
return LiteLLMParamsBody(
|
|
model="openai/gpt-4o-mini",
|
|
api_key="os.environ/OPENAI_API_KEY",
|
|
)
|
|
case "azure":
|
|
return LiteLLMParamsBody(
|
|
model="azure/gpt-5.4-mini-batch",
|
|
api_base="os.environ/AZURE_API_BASE",
|
|
api_key="os.environ/AZURE_API_KEY",
|
|
api_version="2025-04-01-preview",
|
|
)
|
|
case "vertex_ai":
|
|
return LiteLLMParamsBody(
|
|
model="vertex_ai/gemini-2.5-flash",
|
|
vertex_project="os.environ/VERTEXAI_PROJECT",
|
|
vertex_location="us-central1",
|
|
vertex_credentials="os.environ/VERTEXAI_CREDENTIALS",
|
|
gcs_bucket_name="os.environ/GCS_BUCKET_NAME",
|
|
bucket_name="os.environ/GCS_BUCKET_NAME",
|
|
)
|
|
case "bedrock":
|
|
return LiteLLMParamsBody(
|
|
model="bedrock/us.anthropic.claude-haiku-4-5-20251001-v1:0",
|
|
aws_access_key_id="os.environ/AWS_ACCESS_KEY_ID",
|
|
aws_secret_access_key="os.environ/AWS_SECRET_ACCESS_KEY",
|
|
aws_region_name="os.environ/AWS_REGION",
|
|
s3_region_name="os.environ/AWS_REGION",
|
|
s3_bucket_name=_env_ref("AWS_BATCH_S3_BUCKET", "AWS_S3_BUCKET_NAME"),
|
|
s3_access_key_id="os.environ/AWS_ACCESS_KEY_ID",
|
|
s3_secret_access_key="os.environ/AWS_SECRET_ACCESS_KEY",
|
|
aws_batch_role_arn="os.environ/AWS_BATCH_ROLE_ARN",
|
|
)
|
|
case _:
|
|
raise ValueError(f"unknown batch provider: {self.name!r}")
|
|
|
|
|
|
@dataclass(frozen=True, slots=True)
|
|
class Capability:
|
|
provider: str
|
|
model: str
|
|
raw_model: str
|
|
scenario: Scenario
|
|
can_cancel: bool
|
|
can_list: bool
|
|
|
|
@property
|
|
def id(self) -> str:
|
|
return f"{self.provider}-{self.scenario}"
|
|
|
|
@property
|
|
def jsonl_model(self) -> str:
|
|
# Always the provider deployment name. Unified routes via
|
|
# target_model_names; the JSONL body.model must still be a name Azure /
|
|
# Vertex accept. Putting the proxy alias here used to depend on a perfect
|
|
# rewrite, and a stale or mis-selected deployment produced model_not_found.
|
|
return self.raw_model
|
|
|
|
|
|
PROVIDERS: tuple[Provider, ...] = (
|
|
Provider(
|
|
"openai", batch_model_name("openai-batch"), "gpt-4o-mini", can_cancel=True, can_list=True
|
|
),
|
|
Provider(
|
|
"azure",
|
|
batch_model_name("azure-batch"),
|
|
"gpt-5.4-mini-batch",
|
|
can_cancel=True,
|
|
can_list=True,
|
|
),
|
|
Provider(
|
|
"vertex_ai",
|
|
batch_model_name("vertex-batch"),
|
|
"gemini-2.5-flash",
|
|
can_cancel=True,
|
|
can_list=True,
|
|
),
|
|
Provider(
|
|
"bedrock",
|
|
batch_model_name("bedrock-batch"),
|
|
"bedrock/us.anthropic.claude-haiku-4-5-20251001-v1:0",
|
|
can_cancel=False,
|
|
can_list=False,
|
|
),
|
|
)
|
|
|
|
def _model_for(provider_name: str) -> str:
|
|
for provider in PROVIDERS:
|
|
if provider.name == provider_name:
|
|
return provider.model
|
|
raise ValueError(
|
|
f"no batch provider named {provider_name!r} in PROVIDERS; "
|
|
f"known={[p.name for p in PROVIDERS]}"
|
|
)
|
|
|
|
|
|
OPENAI_BATCH_MODEL = _model_for("openai")
|
|
AZURE_BATCH_MODEL = _model_for("azure")
|
|
|
|
BEDROCK_SCENARIOS: tuple[Scenario, ...] = ("unified",)
|
|
|
|
|
|
def scenarios_for_provider(provider: Provider) -> tuple[Scenario, ...]:
|
|
if provider.name == "bedrock":
|
|
return BEDROCK_SCENARIOS
|
|
return SCENARIOS
|
|
|
|
|
|
CAPABILITIES: tuple[Capability, ...] = tuple(
|
|
Capability(p.name, p.model, p.raw_model, scenario, p.can_cancel, p.can_list)
|
|
for p in PROVIDERS
|
|
for scenario in scenarios_for_provider(p)
|
|
)
|
|
|
|
|
|
def raw_id_matches_provider(provider: str, batch_id: str) -> bool:
|
|
if provider in ("openai", "azure"):
|
|
return batch_id.startswith("batch")
|
|
if provider == "vertex_ai":
|
|
return (
|
|
batch_id.startswith("projects/")
|
|
or "batchPredictionJobs" in batch_id
|
|
or batch_id.isdigit()
|
|
)
|
|
if provider == "bedrock":
|
|
return batch_id.startswith("arn:aws:bedrock:")
|
|
return True
|
|
|
|
|
|
FILE_ID_SHAPE: dict[Scenario, IdShape] = {
|
|
"encoded": "model_encoded",
|
|
"unified": "managed",
|
|
"model_param": "raw",
|
|
"provider_fallback": "raw",
|
|
}
|
|
|
|
BATCH_ID_SHAPE: dict[Scenario, IdShape] = {
|
|
"encoded": "model_encoded",
|
|
"unified": "managed",
|
|
"model_param": "model_encoded",
|
|
"provider_fallback": "raw",
|
|
}
|
|
|
|
|
|
def _b64_decode(value: str) -> str:
|
|
padded = value + "=" * (-len(value) % 4)
|
|
try:
|
|
return base64.urlsafe_b64decode(padded).decode()
|
|
except Exception:
|
|
return ""
|
|
|
|
|
|
def is_managed_id(id_str: str) -> bool:
|
|
return _b64_decode(id_str).startswith("litellm_proxy")
|
|
|
|
|
|
def is_model_encoded_id(id_str: str) -> bool:
|
|
for prefix in ("file-", "batch_"):
|
|
if id_str.startswith(prefix):
|
|
decoded = _b64_decode(id_str[len(prefix) :])
|
|
return decoded.startswith("litellm:") and ";model," in decoded
|
|
return False
|
|
|
|
|
|
def matches_id_shape(shape: IdShape, id_str: str) -> bool:
|
|
if shape == "managed":
|
|
return is_managed_id(id_str)
|
|
if shape == "model_encoded":
|
|
return is_model_encoded_id(id_str)
|
|
return not is_managed_id(id_str) and not is_model_encoded_id(id_str)
|
|
|
|
|
|
def coverage_cells_for_lifecycle(cap: Capability) -> tuple[str, ...]:
|
|
"""Registry cell ids that the parametrized lifecycle test covers for one capability.
|
|
|
|
OpenAI has per-scenario cells plus granular create/retrieve/cancel/list/file
|
|
cells. Other providers have one basic cell each. File-upload cells for the
|
|
batch-backing path are included when the lifecycle uploads for that provider.
|
|
"""
|
|
match cap.provider:
|
|
case "openai":
|
|
cells = (
|
|
f"llm.batches.openai_{cap.scenario}.basic.nonstream.works",
|
|
"llm.batches.openai.create.nonstream.works",
|
|
"llm.batches.openai.retrieve.nonstream.works",
|
|
"llm.batches.openai.file_lifecycle.nonstream.works",
|
|
"llm.files.openai.upload.nonstream.works",
|
|
)
|
|
if cap.can_cancel:
|
|
cells = (*cells, "llm.batches.openai.cancel.nonstream.works")
|
|
if cap.can_list:
|
|
cells = (*cells, "llm.batches.openai.list.nonstream.works")
|
|
return cells
|
|
case "azure":
|
|
return (
|
|
"llm.batches.azure_openai.basic.nonstream.works",
|
|
"llm.files.azure_openai.upload.nonstream.works",
|
|
)
|
|
case "vertex_ai":
|
|
return (
|
|
"llm.batches.vertex.basic.nonstream.works",
|
|
"llm.files.vertex.upload.nonstream.works",
|
|
)
|
|
case "bedrock":
|
|
return (
|
|
"llm.batches.bedrock.basic.nonstream.works",
|
|
"llm.files.bedrock.upload.nonstream.works",
|
|
)
|
|
case _:
|
|
return ()
|