Skip to content

Fetch IBM instance profiles #5003

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 26, 2025
Merged

Fetch IBM instance profiles #5003

merged 1 commit into from
Mar 26, 2025

Conversation

vvidovic
Copy link
Contributor

@vvidovic vvidovic commented Mar 20, 2025

Create catalog (ibm/vms.csv) file from the IBM instance profiles fetched by the API.

Unfortunately, it seems that currently, it is not possible to fetch pricing. There is the pricing API but it doesn't work for the kind:instance.profile. It works for the kind:service as described in the IBM documentation (https://cloud.ibm.com/docs/account?topic=account-getting-pricing-api).

Tested (run the relevant ones):

  • Code formatting: install pre-commit (auto-check on commit) or bash format.sh
  • Any manual or new tests for this PR (please specify below)
  • All smoke tests: /smoke-test (CI) or pytest tests/test_smoke.py (local)
  • Relevant individual tests: /smoke-test -k test_name (CI) or pytest tests/test_smoke.py::test_name (local)
  • Backward compatibility: /quicktest-core (CI) or pytest tests/smoke_tests/test_backward_compat.py (local)

@vvidovic
Copy link
Contributor Author

Need to fix some formatting issues.

@vvidovic vvidovic force-pushed the master branch 4 times, most recently from 90ebcb0 to 8d38f22 Compare March 25, 2025 08:22
Create catalog (`ibm/vms.csv`) file from the IBM instance
profiles fetched by the API.
Unfortunately, it seems that currently, it is not possible
to fetch pricing. There is the pricing API but it doesn't
work for the `kind:instance.profile`. It works for the
`kind:service` as described in the IBM documentation
(https://cloud.ibm.com/docs/account?topic=account-getting-pricing-api).
@vvidovic
Copy link
Contributor Author

Need to fix some formatting issues.

All formatting and lint issues are cleared. Please check to include this code for manual IBM instance fetching purposes (pricing fetching is unfortunately not working by IBM API calls).

@cg505 cg505 requested a review from Michaelvll March 25, 2025 18:49
Copy link
Collaborator

@Michaelvll Michaelvll left a comment

Choose a reason for hiding this comment

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

Thanks for adding this @vvidovic! LGTM.

@cg505 cg505 merged commit 349536b into skypilot-org:master Mar 26, 2025
18 checks passed
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.

4 participants