adding step_exception to provide more signal of why the step failed (#730)

This commit is contained in:
Kerem Yilmaz
2024-08-28 11:06:04 +03:00
committed by GitHub
parent 6f4ff4339f
commit a6a545224a
4 changed files with 14 additions and 3 deletions

View File

@@ -14,6 +14,7 @@ from skyvern.forge.sdk.api.llm.exceptions import (
DuplicateCustomLLMProviderError,
InvalidLLMConfigError,
LLMProviderError,
LLMProviderErrorRetryableTask,
)
from skyvern.forge.sdk.api.llm.models import LLMAPIHandler, LLMConfig, LLMRouterConfig
from skyvern.forge.sdk.api.llm.utils import llm_messages_builder, parse_api_response
@@ -211,8 +212,8 @@ class LLMAPIHandlerFactory:
**active_parameters,
)
LOG.info("LLM API call successful", llm_key=llm_key, model=llm_config.model_name)
except openai.OpenAIError as e:
raise LLMProviderError(llm_key) from e
except litellm.exceptions.APIError as e:
raise LLMProviderErrorRetryableTask(llm_key) from e
except CancelledError:
t_llm_cancelled = time.perf_counter()
LOG.error(

View File

@@ -40,6 +40,11 @@ class LLMProviderError(BaseLLMError):
super().__init__(f"Error while using LLMProvider {llm_key}")
class LLMProviderErrorRetryableTask(LLMProviderError):
def __init__(self, llm_key: str) -> None:
super().__init__(f"Retryable error while using LLMProvider {llm_key}")
class NoProviderEnabledError(BaseLLMError):
def __init__(self) -> None:
super().__init__(