[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
Draft
[remove endpoint] fix: remove cloudos job cost command; confirm archive teamId param (api-server#5665)#364danielboloc with Copilot wants to merge 2 commits into
cloudos job cost command; confirm archive teamId param (api-server#5665)#364danielboloc with Copilot wants to merge 2 commits into
Conversation
…archive teamId param
Copilot
AI
changed the title
[WIP] Sync cloudos-cli with CloudOS API endpoint changes
fix: remove Jun 19, 2026
cloudos job cost command; confirm archive teamId param (api-server#5665)
cloudos job cost command; confirm archive teamId param (api-server#5665)cloudos job cost command; confirm archive teamId param (api-server#5665)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Syncs cloudos-cli with two endpoint changes from api-server PR #5665 (
54324c6f).Overview
GET /api/v1/jobs/{job_id}/costs/computewas removed from the API → dropscloudos job costcommand and its implementationPUT /api/v1/jobsspec now formally requiresteamIdquery param → existing code already sends it; no functional change neededJIRA
Changes
@job.command('cost')fromcloudos_cli/jobs/cli.pyand itsCostViewerimportget_job_costsanddisplay_costsfromcloudos_cli/cost/cost.py; retains standalone utility methods (_calculate_runtime,_format_storage,_format_price,_format_lifecycle_type)tests/test_cost/test_job_cost.py: drops tests for deleted methods; addstest_get_job_costs_not_availableandtest_display_costs_not_availableasserting the removed interface is gone2.91.0 → 2.91.1; adds CHANGELOG entryAcceptance Criteria
Scenario 1 — `cloudos job cost` no longer listed
Scenario 2 — archive/unarchive still sends teamId correctly
PUT /api/v1/jobs?teamId=<workspace_id>still issued by bothcloudos job archiveandcloudos 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