Skip to content

Apply cb-spider:0.12.5 and its API changes#2250

Merged
cb-github-robot merged 1 commit intocloud-barista:mainfrom
seokho-son:main
Jan 2, 2026
Merged

Apply cb-spider:0.12.5 and its API changes#2250
cb-github-robot merged 1 commit intocloud-barista:mainfrom
seokho-son:main

Conversation

@seokho-son
Copy link
Copy Markdown
Member

Apply cb-spider:0.12.5 and its API changes

v0.12.3

API Changes
Get VM Price Info:

  • Changed HTTP method: POST → GET
  • Removed filterList parameter from request body
  • Now uses ConnectionName as query parameter only

Also, this PR enhance init.py progress bar

Signed-off-by: Seokho Son <shsongist@gmail.com>
Copilot AI review requested due to automatic review settings January 2, 2026 08:56
@seokho-son seokho-son requested a review from yunkon-kim as a code owner January 2, 2026 08:56
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR upgrades the cb-spider dependency from version 0.12.0 to 0.12.5 and adapts the codebase to accommodate API changes introduced in the new version. The main API change involves the VM price info endpoint switching from POST to GET with query parameters instead of a request body.

Key changes:

  • Updated cb-spider Docker image from 0.12.0 to 0.12.5
  • Refactored price lookup API call to use GET method with query parameters
  • Enhanced init.py progress indicator with a smoother spinner and better TTY/non-TTY handling

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File Description
docker-compose.yaml Updates cb-spider Docker image version to 0.12.5
src/core/resource/spec.go Adapts LookupPriceList function to use GET method with ConnectionName and region as query parameters instead of POST with request body
init/init.py Replaces progress bar with spinner animation, adds TTY detection for better display in different environments, and increases default expected completion time to 400 seconds

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@seokho-son
Copy link
Copy Markdown
Member Author

/approve

@github-actions github-actions bot added the approved This PR is approved and will be merged soon. label Jan 2, 2026
@cb-github-robot cb-github-robot merged commit 5f1b77f into cloud-barista:main Jan 2, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved This PR is approved and will be merged soon.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants