feat(BRE2-661): Add Support for San Francisco Compute#87
Draft
feat(BRE2-661): Add Support for San Francisco Compute#87
Conversation
* add scaffolding, and client authentication.
* return APITypeGlobal from GetAPIType function, as SFC accounts are not tied to specific regions.
* fix apiKey in SFCCredential struct
* scaffolding for instance.go
* add instance creation implementation with SSH key support
* add function to map the status of a node reported from SFC API to v1.LifecycleStatus in Brev
* implement GetInstance in sfcompute with node data retrieval inluding SSH Hostname / Public IP
* add TerminateInstance implementation with node release and delete logic
* set default SSH port to 2222, as is standard for our platform
* implement GetSSHHostname for retrieving the SSH hostname of an instance in sfcompute
* remove unneeded call to api for GetSSHHostname
* use VM ID instead of instance ID to retrieve SSH Hostname
* remove get ssh hostname function
* implement ListInstances
* add validation test for sfcompute with API key check and skip logic
* add getInstanceTypeID method for generating instance type IDs in sfcompute
* bump sfcnodes version to v0.1.0-alpha.4 which adds support for the /v0/zones endpoint
* implement GetLocations
* only return approved zones
* only return regions that have more than zero capacity instead of any zones that have capacity not equal to zero, in case the availability ever returns a negative number
* update location description to include formatted hardware type information. example: `sfc_hayesvalley_h100`
* return unavailable regions with v1.Location{Available: false}
* fix an error where a nil map was returned
* start implementing GetInstanceTypes
* fix tests failing due to ValidateRegionalInstanceTypes and ValidateStableInstanceTypeIDs fails errors
* set MaxPricePerNodePerHour to 1600 ($16/node/h, $2/gpu/h)
* add regions excelsior and yerba
* remove region excelsior
drewmalin
changed the title
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
2 participants
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.
No description provided.