Skip to content

Conversation

Bihan
Copy link
Collaborator

@Bihan Bihan commented Aug 26, 2025

Steps

  1. Login to both clouds https://cloud.digitalocean.com/login and https://amd.digitalocean.com/login

  2. To configure backends add

  - type: amddevcloud
    project_name: TestProject
    creds:
      type: api_key
      api_key: <api_key_amd_dev_cloud>
  - type: digitalocean
    project_name: TestProjectNvidia
    creds:
      type: api_key
      api_key: <api_key_digitalocean>

Note: Will be updating the PR with project_name configuration. Creation of Droplet (VM) was not documented in api reference, but recently found that "project_id" field works.

@Bihan Bihan requested review from jvstme and peterschmidt85 August 26, 2025 10:21
Copy link
Collaborator

@jvstme jvstme left a comment

Choose a reason for hiding this comment

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

@Bihan, thanks for the PR. The division into two backends looks good to me 👍 Please also see my comments about some implementation details

Copy link
Collaborator

@jvstme jvstme left a comment

Choose a reason for hiding this comment

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

@Bihan, thanks for the update. I think we are almost ready to merge. The remaining things are:

  • Bump gpuhunt to 0.1.8 in pyproject.toml
  • Address my non-nitpicking comments on the latest update
  • Optionally address my nitpicking comments

If you decide to follow my code suggestions and there will be no details to discuss, feel free to merge without repeating the review. Thank you!

Bihan and others added 6 commits September 2, 2025 16:23
…t.py

Co-authored-by: jvstme <36324149+jvstme@users.noreply.github.com>
Co-authored-by: jvstme <36324149+jvstme@users.noreply.github.com>
Co-authored-by: jvstme <36324149+jvstme@users.noreply.github.com>
Co-authored-by: jvstme <36324149+jvstme@users.noreply.github.com>
Co-authored-by: jvstme <36324149+jvstme@users.noreply.github.com>
@Bihan Bihan merged commit b375683 into dstackai:master Sep 2, 2025
27 of 56 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.

2 participants