Skip to content

Commit 1b7c6fd

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

File tree

7 files changed

+594
-799
lines changed

7 files changed

+594
-799
lines changed

go.sum

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -298,8 +298,6 @@ github.com/robfig/cron/v3 v3.0.1 h1:WdRxkvbJztn8LMz/QEvLN5sBU+xKpSqwwUO1Pjr4qDs=
298298
github.com/robfig/cron/v3 v3.0.1/go.mod h1:eQICP3HwyT7UooqI/z+Ov+PtYAWygg1TEWWzGIFLtro=
299299
github.com/rogpeppe/go-internal v1.13.1 h1:KvO1DLK/DRN07sQ1LQKScxyZJuNnedQ5/wKSR38lUII=
300300
github.com/rogpeppe/go-internal v1.13.1/go.mod h1:uMEvuHeurkdAXX61udpOXGD/AzZDWNMNyH2VO9fmH0o=
301-
github.com/scaleway/scaleway-sdk-go v1.0.0-beta.33.0.20250424152954-b4babe8f214c h1:sjbNFhI3o5ecQuxLZv54Gm/YlqP55Ot5l7ShneWeNg8=
302-
github.com/scaleway/scaleway-sdk-go v1.0.0-beta.33.0.20250424152954-b4babe8f214c/go.mod h1:w4o02EHpO0CBGy2nehzWRaFQKd62G9HIf+Q07PDaUcE=
303301
github.com/scaleway/scaleway-sdk-go v1.0.0-beta.33.0.20250425085959-ea0a849e0b26 h1:6KJ16mZbrP/ahxkbJGTCjHdJJdCJF1Hfwnw92Q5sf3I=
304302
github.com/scaleway/scaleway-sdk-go v1.0.0-beta.33.0.20250425085959-ea0a849e0b26/go.mod h1:w4o02EHpO0CBGy2nehzWRaFQKd62G9HIf+Q07PDaUcE=
305303
github.com/sergi/go-diff v1.3.2-0.20230802210424-5b0b94c5c0d3 h1:n661drycOFuPLCN3Uc8sB6B/s6Z4t2xvBgU1htSHuq8=

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 && && !httperrors.Is404(err) {
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
}

internal/services/inference/testdata/custom-model-basic.cassette.yaml

Lines changed: 46 additions & 46 deletions
Large diffs are not rendered by default.

internal/services/inference/testdata/custom-model-deploy-model-on-server.cassette.yaml

Lines changed: 165 additions & 263 deletions
Large diffs are not rendered by default.

internal/services/inference/testdata/deployment-basic.cassette.yaml

Lines changed: 123 additions & 221 deletions
Large diffs are not rendered by default.

internal/services/inference/testdata/deployment-endpoint.cassette.yaml

Lines changed: 247 additions & 247 deletions
Large diffs are not rendered by default.

0 commit comments

Comments
 (0)