Skip to content

Conversation

@cardosofede
Copy link
Contributor

No description provided.

@rapcmia
Copy link
Contributor

rapcmia commented Jan 28, 2026

Commit e01b64a

  • Setup this PR with docker build on Ubuntu24.04
  • Tested with position_executor
  • DELETE /executors/{executor_id} has been removed
  • POST /executors, /executor/search, /executors/{executor_id}/stop
  • GET /executors/{executor_id}, GET /executors/types/available and GET /executors/types/{executor_type}/config
  • Found bug on GET /executors/positions/summary and /executors/positions/{connector_name}/{trading_pair
  • I compared the issues found when testing this PR with main branch

GET /executors/summary, Error getting executor summary: 1 validation error for ExecutorsSummaryResponse ❌


2026-01-28 06:06:32,284 - routers.executors - ERROR - Error getting executor summary: 1 validation error for ExecutorsSummaryResponse
total_completed
  Field required [type=missing, input_value={'total_active': 1, 'tota...status': {'RUNNING': 1}}, input_type=dict]
    For further information visit https://errors.pydantic.dev/2.12/v/missing
Traceback (most recent call last):
  File "/hummingbot-api/routers/executors.py", line 153, in get_executors_summary
    return ExecutorsSummaryResponse(**summary)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/conda/envs/hummingbot-api/lib/python3.12/site-packages/pydantic/main.py", line 250, in __init__
    validated_self = self.__pydantic_validator__.validate_python(data, self_instance=self)
                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
pydantic_core._pydantic_core.ValidationError: 1 validation error for ExecutorsSummaryResponse
total_completed
  Field required [type=missing, input_value={'total_active': 1, 'tota...status': {'RUNNING': 1}}, input_type=dict]
    For further inforation visit https://errors.pydantic.dev/2.12/v/missing
06:06:32.282 GET /executors/summary
  • Steps to reproduce: Open a executor and check executor/summary

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants