Skip to content

fix: bundle vikingdb wheel for byteplus studio - #1145

Merged
yaozheng-fang merged 1 commit into
volcengine:mainfrom
xgtcode:fix/byteplus-vikingdb-wheel
Sep 22, 2026
Merged

yaozheng-fang merged 1 commit into
volcengine:mainfrom
xgtcode:fix/byteplus-vikingdb-wheel

Conversation

@xgtcode

@xgtcode xgtcode commented Sep 22, 2026

Copy link
Copy Markdown
Collaborator

Summary

Fix BytePlus Studio source/update deployments by bundling the pinned vikingdb-python-sdk==0.1.32 wheel in the BytePlus-specific Studio dependency set.

BytePlus package resolution can currently fail because veadk-python requires vikingdb-python-sdk>=0.1.32, while the BytePlus install source may only expose versions up to 0.1.26. By staging the 0.1.32 wheel locally in the Studio package, the cloud-side install no longer depends on BytePlus index freshness for this dependency.

Changes

  • Add vikingdb_python_sdk-0.1.32-py3-none-any.whl to BYTEPLUS_STUDIO_DEPENDENCY_WHEELS
  • Add a regression test confirming the VikingDB wheel is included for BytePlus but not Volcengine
  • Update Studio deploy test fixtures to reflect the new BytePlus dependency set

Verification

  • uv run --frozen --group dev pre-commit run --files veadk/cli/studio_dependencies.py tests/cli/test_studio_release.py tests/cli/test_studio_deploy_target.py
  • uv run --frozen --group dev pytest tests/cli/test_studio_release.py tests/cli/test_studio_deploy_target.py -q
    • 97 passed
  • uv run --frozen --group dev pytest tests/cli/test_studio_sidecar_prerequisites.py tests/cli/test_studio_update.py -q
    • 53 passed, 4 warnings

@yaozheng-fang
yaozheng-fang merged commit 6303f37 into volcengine:main Sep 22, 2026
11 checks passed
PolarisCzh pushed a commit to PolarisCzh/veadk-python that referenced this pull request Sep 29, 2026
Co-authored-by: shanchunhua <shanchunhua@bytedance.com>
Co-authored-by: TRAE CLI <traecli@bytedance.com>
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