Skip to content

Refine ResourceManagerV1 telemetry stubs - #39

Merged
xunyoyo merged 1 commit into
fastdeploy/engine/sched/resource_manager_v1.pyfrom
codex/fix-ci-errors-for-resourcemanagerv1-tests
Nov 27, 2025
Merged

xunyoyo merged 1 commit into
fastdeploy/engine/sched/resource_manager_v1.pyfrom
codex/fix-ci-errors-for-resourcemanagerv1-tests

Conversation

@xunyoyo

@xunyoyo xunyoyo commented Nov 27, 2025

Copy link
Copy Markdown
Owner

Summary

  • replace broad OpenTelemetry shims with a minimal fastdeploy.metrics.trace_util stub to satisfy ResourceManagerV1 imports
  • ensure real fastdeploy engine and cache_manager modules are loaded so tests can patch main_process_metrics and cache stubs reliably

Testing

  • pytest -c tests/cov_pytest.ini tests/engine/test_resource_manager_v1.py -vv
  • python -m coverage run -m pytest -c tests/cov_pytest.ini tests/engine/test_resource_manager_v1.py
  • python -m coverage report -m --include='fastdeploy/engine/sched/resource_manager_v1.py'
  • pre-commit run --all-files

Codex Task

@xunyoyo
xunyoyo merged commit 98dcb65 into fastdeploy/engine/sched/resource_manager_v1.py Nov 27, 2025
@xunyoyo
xunyoyo deleted the codex/fix-ci-errors-for-resourcemanagerv1-tests branch December 23, 2025 06:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant