Skip to content

[SDK] Drop CServe create/update/recipe methods - #159

Merged
michaelshin merged 1 commit into
mainfrom
atokayev/drop-cserve
Nov 13, 2026
Merged

michaelshin merged 1 commit into
mainfrom
atokayev/drop-cserve

Conversation

@atokayev

Copy link
Copy Markdown
Contributor

Why

The platform removed the CServe v3 request models (CentML/platform#4881) and CServe create/update now return HTTP 410. centml/sdk/api.py hard-imports CreateCServeV3DeploymentRequest, so import centml.sdk.api raises ImportError against any platform-api-python-client newer than 4.31.0 (4.37.1 is merged in platform_api_python_client but not yet released to PyPI). The pin to 4.31.0 hides it for users, but it breaks anything that overlays the newer client, such as the platform e2e suite.

What

  • Remove create_cserve, update_cserve, get_cserve_recipe and the CreateCServeV3DeploymentRequest import.
  • Delete examples/sdk/create_cserve.py.
  • Keep get_cserve and CLI display of existing CServe deployments (DeploymentType.CSERVE* still exist in the client).
  • Test fixture: add the now-required cost_per_hr to HardwareInstanceResponse (needed for 4.37.x, ignored by 4.31.0).

Validation

pytest tests: 119 passed against platform-api-python-client 4.31.0 and against 4.37.1 built from platform_api_python_client main. On main, import centml.sdk.api fails with the ImportError against 4.37.1.

Not included

requirements.txt still pins 4.31.0. Bumping it needs a platform_api_python_client release newer than v4.31.0 (none since 2026-09-23, so PyPI has no 4.32+); follow-up once that is published.

Users calling create_cserve/update_cserve/get_cserve_recipe lose those methods; they already receive HTTP 410 from the API.

🤖 Generated with Claude Code

…orm API

The platform removed the CServe v3 request models and now answers CServe create/update
with HTTP 410, so 'from platform_api_python_client import CreateCServeV3DeploymentRequest'
fails against clients newer than 4.31.0. Remove the create_cserve, update_cserve and
get_cserve_recipe methods and the create_cserve example. Reads (get_cserve) and the CLI
display of existing CServe deployments are unchanged.

Also supply the now-required cost_per_hr in a HardwareInstanceResponse test fixture.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@michaelshin
michaelshin merged commit b3e18dc into main Nov 13, 2026
6 checks passed
@michaelshin
michaelshin deleted the atokayev/drop-cserve branch November 13, 2026 18: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.

2 participants