Skip to content

Fix BosClient upload API compatibility#3242

Open
ShigureNyako wants to merge 1 commit into
PaddlePaddle:developfrom
ShigureNyako:fix/bosclient-upload-api-compatibility
Open

Fix BosClient upload API compatibility#3242
ShigureNyako wants to merge 1 commit into
PaddlePaddle:developfrom
ShigureNyako:fix/bosclient-upload-api-compatibility

Conversation

@ShigureNyako

Copy link
Copy Markdown

Motivation

Paddle Model-Benchmark 流水线 使用新版 bce-python-sdk 时,BosClient 只提供拼写正确的 put_super_object_from_file;但 PaddleTest 的部分 workflow 仍固定在只提供旧 typo API put_super_obejct_from_file 的版本。

因此上传脚本需要同时兼容两代 API。

Changes

  • tools/bos_tools.pytools/bos_upload.pypublish_scripts/PypiIndexUpdate.py 增加兼容方法解析。
  • 优先调用正式 API put_super_object_from_file,缺失时回退到旧 API put_super_obejct_from_file
  • 保持现有上传参数及 SDK 版本约束不变。

Validation

  • git diff --check
  • python3 -m compileall -q tools/bos_tools.py tools/bos_upload.py publish_scripts/PypiIndexUpdate.py
  • 使用 fake old/new/dual clients 验证三个兼容 helper,包含“同时存在时优先正式 API”的场景。
  • 使用 PyPI 官方 wheel 中的真实 BosClient 验证:
    • bce-python-sdk==0.8.740.9.29 解析到旧 typo API;
    • bce-python-sdk==0.9.670.9.72 解析到正式 API。

未执行真实 BOS 上传:该操作需要 AK/SK 凭据,并会产生外部写入。

Related Issues

@ShigureNyako

Copy link
Copy Markdown
Author

已完成自查:改动仅包含 3 个独立 BOS 上传脚本的 API 兼容处理,旧版与新版 SDK 的方法解析验证均已通过。@SigureMo 麻烦 review,感谢!

Co-authored-by: SigureMo <SigureMo@users.noreply.github.com>
Co-authored-by: Codex <noreply@openai.com>
@ShigureNyako
ShigureNyako force-pushed the fix/bosclient-upload-api-compatibility branch from f078020 to 613697c Compare July 20, 2026 17:47
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.

1 participant