Skip to content

Commit c68fb90

Browse files
committed
use of dsf.locality
1 parent 86fcc05 commit c68fb90

File tree

6 files changed

+594
-797
lines changed

6 files changed

+594
-797
lines changed

internal/services/inference/custom_model.go

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -219,8 +219,12 @@ func ResourceCustomModelDelete(ctx context.Context, d *schema.ResourceData, m in
219219
return diag.FromErr(err)
220220
}
221221

222-
return diag.FromErr(api.DeleteModel(&inference.DeleteModelRequest{
222+
if err := api.DeleteModel(&inference.DeleteModelRequest{
223223
Region: region,
224-
ModelID: d.Id(),
225-
}, scw.WithContext(ctx)))
224+
ModelID: id,
225+
}, scw.WithContext(ctx)); err != nil {
226+
return diag.FromErr(err)
227+
}
228+
229+
return nil
226230
}

internal/services/inference/deployment.go

Lines changed: 6 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ import (
99
"github.com/scaleway/scaleway-sdk-go/api/inference/v1"
1010
"github.com/scaleway/scaleway-sdk-go/scw"
1111
scwvalidation "github.com/scaleway/scaleway-sdk-go/validation"
12+
"github.com/scaleway/terraform-provider-scaleway/v2/internal/dsf"
1213
"github.com/scaleway/terraform-provider-scaleway/v2/internal/httperrors"
1314
"github.com/scaleway/terraform-provider-scaleway/v2/internal/locality/regional"
1415
"github.com/scaleway/terraform-provider-scaleway/v2/internal/services/account"
@@ -52,22 +53,11 @@ func ResourceDeployment() *schema.Resource {
5253
Description: "The model name to use for the deployment",
5354
},
5455
"model_id": {
55-
Type: schema.TypeString,
56-
Required: true,
57-
Description: "The model id used for the deployment",
58-
ForceNew: true,
59-
DiffSuppressFunc: func(k, old, newValue string, d *schema.ResourceData) bool {
60-
if old == "" || newValue == "" {
61-
return false
62-
}
63-
if !scwvalidation.IsUUID(newValue) {
64-
newID := regional.ExpandID(newValue).ID
65-
66-
return old == newID
67-
}
68-
69-
return old == newValue
70-
},
56+
Type: schema.TypeString,
57+
Required: true,
58+
Description: "The model id used for the deployment",
59+
ForceNew: true,
60+
DiffSuppressFunc: dsf.Locality,
7161
},
7262
"accept_eula": {
7363
Type: schema.TypeBool,
@@ -197,7 +187,6 @@ func ResourceDeploymentCreate(ctx context.Context, d *schema.ResourceData, m int
197187
}
198188

199189
modelID := d.Get("model_id")
200-
201190
if !scwvalidation.IsUUID(modelID.(string)) {
202191
modelID = regional.ExpandID(modelID).ID
203192
}

0 commit comments

Comments
 (0)