Skip to content

Commit 9e246d6

Browse files
author
Andrei Neagu
committed
fixed error formatting
1 parent 5e31621 commit 9e246d6

File tree

1 file changed

+11
-4
lines changed
  • packages/service-library/src/servicelib/long_running_tasks/_rabbit

1 file changed

+11
-4
lines changed

packages/service-library/src/servicelib/long_running_tasks/_rabbit/lrt_client.py

Lines changed: 11 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
from models_library.rabbitmq_basic_types import RPCMethodName
66
from pydantic import PositiveInt, TypeAdapter
77

8+
from ...logging_errors import create_troubleshootting_log_kwargs
89
from ...logging_utils import log_decorator
910
from ...long_running_tasks.task import RegisteredTaskName
1011
from ...rabbitmq._client_rpc import RabbitMQRPCClient
@@ -104,10 +105,16 @@ async def get_task_result(
104105
if isinstance(serialized_result, RPCErrorResponse):
105106
error = string_to_object(serialized_result.error_object)
106107
_logger.warning(
107-
"Remote task finished with error: '%s: %s'\n%s",
108-
error.__class__.__name__,
109-
error,
110-
serialized_result.str_traceback,
108+
**create_troubleshootting_log_kwargs(
109+
f"Remote task finished with error '{error.__class__.__name__}: {error}'\n{serialized_result.str_traceback}",
110+
error=error,
111+
error_context={
112+
"task_id": task_id,
113+
"task_context": task_context,
114+
"namespace": namespace,
115+
},
116+
tip=f"Raised where the lrt_server was running, you can figure this out via {namespace=}",
117+
)
111118
)
112119
raise error
113120

0 commit comments

Comments
 (0)