Skip to content

Conversation

@habfast
Copy link
Contributor

@habfast habfast commented Nov 20, 2025

Description

shortcut story: sc-151282
Also fixes sc-151025 by bumping egoscale

Checklist

(For exoscale contributors)

  • Changelog updated no changes for the users
  • Testing

Testing

simply created and deleted models and instances.

@habfast habfast requested a review from a team November 20, 2025 16:09
@habfast habfast force-pushed the paul.habfast/sc-151282_aiservices-cli-improvements branch from 4955a0f to 3061ab1 Compare November 20, 2025 20:25
Comment on lines 48 to 54
if err != nil {
return
}
err = r.Body.Close()
if err != nil {
return
}
Copy link
Contributor

Choose a reason for hiding this comment

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

If you want you can t.Fail() here

@@ -0,0 +1,3 @@
package deployment
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's remove the file ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oops I thought I had done it 👀 . Absolutely

Replicas int64 `json:"replicas"`
ServiceLevel string `json:"service_level"`
DeploymentURL string `json:"deployment_url"`
ModelID *v3.UUID `json:"model_id"`
Copy link
Member

Choose a reason for hiding this comment

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

is this one need to be a pointer?

@arapaho arapaho changed the title Paul.habfast/sc-151282 aiservices cli improvements aiservices cli improvements Nov 26, 2025
@habfast habfast merged commit e9b1c05 into master Nov 27, 2025
3 checks passed
@habfast habfast deleted the paul.habfast/sc-151282_aiservices-cli-improvements branch November 27, 2025 14:54
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