Skip to content

feat(sandbox): expose sandbox resource APIs - #473

Merged
miclle merged 2 commits into
qiniu:masterfrom
DROWNING2003:feat/sandbox-resources
Sep 10, 2026
Merged

feat(sandbox): expose sandbox resource APIs#473
miclle merged 2 commits into
qiniu:masterfrom
DROWNING2003:feat/sandbox-resources

Conversation

@DROWNING2003

Copy link
Copy Markdown
Contributor

Summary

  • expose sandbox resource listing and Git repository token updates
  • support inline Kodo credentials and template build disk size

Validation

  • PYTHONPATH=. pytest -q tests/cases/test_services/test_sandbox/test_client.py

Expose sandbox resource management and support inline Kodo credentials. Add template disk size configuration and regression coverage.
@qiniu-prow qiniu-prow Bot added the size/L label Sep 9, 2026

@fennoai fennoai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed the complete fixed diff for sandbox resource listing/token updates, inline Kodo credentials, and template disk-size normalization. The changed request paths, validation, auth selection, aliases, and existing compatibility paths are consistent; I found no actionable correctness or regression issue.

Verification: python -m compileall -q qiniu/services/sandbox tests/cases/test_services/test_sandbox/test_client.py and git diff --check pass. The requested pytest command was not runnable because this environment does not have pytest or the requests dependency installed.

Keep the sandbox client compliant with the repository lint gate across the supported Python versions.
@miclle

miclle commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

LGTM

@miclle
miclle merged commit e1bc537 into qiniu:master Sep 10, 2026
10 checks passed
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.

2 participants