Skip to content

[remove endpoint] fix: remove cloudos job cost command; confirm archive teamId param (api-server#5665)#364

Draft
danielboloc with Copilot wants to merge 2 commits into
mainfrom
copilot/sync-cloudos-cli-api-endpoint-change-again
Draft

[remove endpoint] fix: remove cloudos job cost command; confirm archive teamId param (api-server#5665)#364
danielboloc with Copilot wants to merge 2 commits into
mainfrom
copilot/sync-cloudos-cli-api-endpoint-change-again

Conversation

Copilot AI commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Syncs cloudos-cli with two endpoint changes from api-server PR #5665 (54324c6f).

Overview

  • GET /api/v1/jobs/{job_id}/costs/compute was removed from the API → drops cloudos job cost command and its implementation
  • PUT /api/v1/jobs spec now formally requires teamId query param → existing code already sends it; no functional change needed

JIRA

  • N/A (API contract sync)

Changes

  • Removes @job.command('cost') from cloudos_cli/jobs/cli.py and its CostViewer import
  • Removes get_job_costs and display_costs from cloudos_cli/cost/cost.py; retains standalone utility methods (_calculate_runtime, _format_storage, _format_price, _format_lifecycle_type)
  • Updates tests/test_cost/test_job_cost.py: drops tests for deleted methods; adds test_get_job_costs_not_available and test_display_costs_not_available asserting the removed interface is gone
  • Removes "Get Job Costs" README section and TOC entry
  • Bumps version 2.91.0 → 2.91.1; adds CHANGELOG entry

Acceptance Criteria

Scenario 1 — `cloudos job cost` no longer listed
$ cloudos job --help
# 'cost' command absent from Commands list
Scenario 2 — archive/unarchive still sends teamId correctly

PUT /api/v1/jobs?teamId=<workspace_id> still issued by both cloudos job archive and cloudos job unarchive — no regression.

DEV

Proof this feature/patch works in this environment

AZURE

Proof this feature/patch works in this environment

Interactive Analysis

Proof this feature/patch works in this environment

Copilot AI changed the title [WIP] Sync cloudos-cli with CloudOS API endpoint changes fix: remove cloudos job cost command; confirm archive teamId param (api-server#5665) Jun 19, 2026
Copilot AI requested a review from danielboloc June 19, 2026 13:06
@danielboloc danielboloc changed the title fix: remove cloudos job cost command; confirm archive teamId param (api-server#5665) [remove endpoint] fix: remove cloudos job cost command; confirm archive teamId param (api-server#5665) Jun 19, 2026
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