Skip to content

Add disk_usage to venv2 updated deps - #4056

Merged
joaoantoniocardoso merged 1 commit into
bluerobotics:masterfrom
nicoschmdt:bump-disk
Jul 28, 2026
Merged

Add disk_usage to venv2 updated deps#4056
joaoantoniocardoso merged 1 commit into
bluerobotics:masterfrom
nicoschmdt:bump-disk

Conversation

@nicoschmdt

Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions

Copy link
Copy Markdown

Automated PR Review

0. Summary

  • Verdict: LGTM :shipit:

Moves the disk_usage service from venv1 to venv2: removed from core/pyproject.toml's workspace/sources and added to the exclude list, added to core/python-venv2/pyproject.toml, bumped its runtime deps to the venv2 baseline (fastapi==0.125.0, fastapi-versioning==0.10.0, loguru==0.7.3, pydantic==2.12.5, uvicorn==0.38.0), migrated the two pydantic v1 idioms in main.py (update_forward_refs()model_rebuild(), .dict().model_dump()), and updated core/start-blueos-core to launch it under $BLUEOS_PYTHON_BIN_SECONDARY. Lock files regenerated accordingly.

Version pins now match the existing venv2 services (customization/helper/kraken/versionchooser) exactly, both pydantic v1 call sites in main.py are converted, and no other v1-only patterns (.parse_obj, .parse_raw, Config:, @validator, .json()) remain in the service. nginx routing to port 9151 is untouched.

No further comments, nice job 👍

Generated by PR Review Bot. This is advisory, a human reviewer must still approve.

@joaoantoniocardoso
joaoantoniocardoso merged commit beb3f11 into bluerobotics:master Jul 28, 2026
8 checks passed
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.

2 participants