diff --git a/internal/services/instance/ip.go b/internal/services/instance/ip.go index fa372b5f8..94769d122 100644 --- a/internal/services/instance/ip.go +++ b/internal/services/instance/ip.go @@ -147,7 +147,7 @@ func ResourceInstanceIPRead(ctx context.Context, d *schema.ResourceData, m any) Zone: zone, }, scw.WithContext(ctx)) if err != nil { - // We check for 403 because instanceSDK API returns 403 for a deleted IP + // We check for 403 because instance API returns 403 for a deleted IP if httperrors.Is404(err) || httperrors.Is403(err) { d.SetId("") @@ -202,7 +202,7 @@ func ResourceInstanceIPDelete(ctx context.Context, d *schema.ResourceData, m any Zone: zone, }, scw.WithContext(ctx)) if err != nil { - // We check for 403 because instanceSDK API returns 403 for a deleted IP + // We check for 403 because instance API returns 403 for a deleted IP if httperrors.Is404(err) || httperrors.Is403(err) { d.SetId("") diff --git a/internal/services/instance/ip_reverse_dns.go b/internal/services/instance/ip_reverse_dns.go index 9e29a8356..6ffdbb372 100644 --- a/internal/services/instance/ip_reverse_dns.go +++ b/internal/services/instance/ip_reverse_dns.go @@ -97,7 +97,7 @@ func ResourceInstanceIPReverseDNSRead(ctx context.Context, d *schema.ResourceDat Zone: zone, }, scw.WithContext(ctx)) if err != nil { - // We check for 403 because instanceSDK API returns 403 for a deleted IP + // We check for 403 because instance API returns 403 for a deleted IP if httperrors.Is404(err) || httperrors.Is403(err) { d.SetId("") diff --git a/internal/services/instance/server.go b/internal/services/instance/server.go index 714febbd2..3aba629ab 100644 --- a/internal/services/instance/server.go +++ b/internal/services/instance/server.go @@ -1189,7 +1189,7 @@ func ResourceInstanceServerDelete(ctx context.Context, d *schema.ResourceData, m if err != nil { return diag.FromErr(err) } - // detach eip to ensure to free eip even if instanceSDK won't stop + // detach eip to ensure to free eip even if instance won't stop if ipID, ok := d.GetOk("ip_id"); ok { _, err := api.UpdateIP(&instanceSDK.UpdateIPRequest{ Zone: zone, @@ -1200,7 +1200,7 @@ func ResourceInstanceServerDelete(ctx context.Context, d *schema.ResourceData, m log.Print("[WARN] Failed to detach eip of server") } } - // Remove instanceSDK from placement group to free it even if instanceSDK won't stop + // Remove instance from placement group to free it even if instance won't stop if _, ok := d.GetOk("placement_group_id"); ok { _, err := api.UpdateServer(&instanceSDK.UpdateServerRequest{ Zone: zone, diff --git a/internal/services/instance/server_test.go b/internal/services/instance/server_test.go index f4bf91da7..87dccc196 100644 --- a/internal/services/instance/server_test.go +++ b/internal/services/instance/server_test.go @@ -954,7 +954,7 @@ func arePrivateNICsPresent(tt *acctest.TestTools, n string) resource.TestCheckFu } // serverHasNewVolume tests if volume name is generated by terraform -// It is useful as volume should not be set in request when creating an instanceSDK from an image +// It is useful as volume should not be set in request when creating an instance from an image func serverHasNewVolume(_ *acctest.TestTools, n string) resource.TestCheckFunc { return func(s *terraform.State) error { rs, ok := s.RootModule().Resources[n] diff --git a/internal/services/instance/testfuncs/checks.go b/internal/services/instance/testfuncs/checks.go index b6431673e..81a6f7152 100644 --- a/internal/services/instance/testfuncs/checks.go +++ b/internal/services/instance/testfuncs/checks.go @@ -147,7 +147,7 @@ func IsIPDestroyed(tt *acctest.TestTools) resource.TestCheckFunc { case err == nil: return retry.RetryableError(fmt.Errorf("instance IP (%s) still exists", rs.Primary.ID)) // Unexpected api error we return it - // We check for 403 because instanceSDK API return 403 for deleted IP + // We check for 403 because instance API return 403 for deleted IP case httperrors.Is404(err) || httperrors.Is403(err): continue default: