fix(lb): avoid recreation of old lb ips (scaleway#2556)

* fix(lb): avoid recreation of lb ips

* update cassette

Author: yfodil

internal/services/lb/ip.go

@@ -2,6 +2,7 @@ package lb
 
 import (
 	"context"
+	"net"
 
 	"github.com/hashicorp/terraform-plugin-sdk/v2/diag"
 	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/retry"
@@ -139,6 +140,15 @@ func resourceLbIPRead(ctx context.Context, d *schema.ResourceData, m interface{}
 	_ = d.Set("reverse", ip.Reverse)
 	_ = d.Set("lb_id", types.FlattenStringPtr(ip.LBID))
 
+	isIPv6 := false
+	if ip.IPAddress != "" {
+		parsedIP := net.ParseIP(ip.IPAddress)
+		if parsedIP != nil && parsedIP.To4() == nil {
+			isIPv6 = true
+		}
+	}
+	_ = d.Set("is_ipv6", isIPv6)
+
 	return nil
 }
 

internal/services/lb/ip_test.go

@@ -30,6 +30,7 @@ func TestAccIP_Basic(t *testing.T) {
 					isIPPresent(tt, "scaleway_lb_ip.ipZone"),
 					acctest.CheckResourceAttrIPv4("scaleway_lb_ip.ipZone", "ip_address"),
 					resource.TestCheckResourceAttrSet("scaleway_lb_ip.ipZone", "reverse"),
+					resource.TestCheckResourceAttr("scaleway_lb_ip.ipZone", "is_ipv6", "false"),
 					resource.TestCheckResourceAttr("scaleway_lb_ip.ipZone", "zone", "nl-ams-1"),
 				),
 			},
@@ -95,6 +96,7 @@ func TestAccIP_IPv6(t *testing.T) {
 				Check: resource.ComposeTestCheckFunc(
 					isIPPresent(tt, "scaleway_lb_ip.ipv6"),
 					acctest.CheckResourceAttrIPv6("scaleway_lb_ip.ipv6", "ip_address"),
+					resource.TestCheckResourceAttr("scaleway_lb_ip.ipv6", "is_ipv6", "true"),
 				),
 			},
 		},