From f3f8c219dd45a85bd22ef83b3ea8e5cdca66ce1d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Le=C3=AFla=20Marabese?= Date: Thu, 6 Nov 2025 18:38:31 +0100 Subject: [PATCH] doc(instance): fix typos in comments --- internal/services/instance/ip.go | 4 ++-- internal/services/instance/ip_reverse_dns.go | 2 +- internal/services/instance/server.go | 4 ++-- internal/services/instance/server_test.go | 2 +- internal/services/instance/testfuncs/checks.go | 2 +- 5 files changed, 7 insertions(+), 7 deletions(-) diff --git a/internal/services/instance/ip.go b/internal/services/instance/ip.go index fa372b5f83..94769d122b 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 9e29a83565..6ffdbb3721 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 714febbd2f..3aba629ab0 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 f4bf91da7d..87dccc196d 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 b6431673ec..81a6f71523 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: