feat: add DiffSuppressFunc to scaleway_domain_record name attribute to accept FQDN and relative names

Author: jremy42

internal/services/domain/helpers.go

@@ -643,3 +643,29 @@ func BuildZoneName(subdomain, domain string) string {
 
 	return fmt.Sprintf("%s.%s", subdomain, domain)
 }
+
+// normalizeRecordName extracts relative name from FQDN or returns the name as-is
+// Example: "_dmarc.example.com." with dnsZone "example.com" returns "_dmarc"
+// Example: "_dmarc" with dnsZone "example.com" returns "_dmarc"
+func normalizeRecordName(name, dnsZone string) string {
+	if name == "" || name == "@" {
+		return ""
+	}
+
+	// Remove trailing dot
+	name = strings.TrimSuffix(name, ".")
+	dnsZone = strings.TrimSuffix(dnsZone, ".")
+
+	// If name equals dns zone, it's root zone
+	if name == dnsZone {
+		return ""
+	}
+
+	// Remove domain suffix if present
+	suffix := "." + dnsZone
+	if strings.HasSuffix(name, suffix) {
+		return strings.TrimSuffix(name, suffix)
+	}
+
+	return name
+}

internal/services/domain/record.go

@@ -73,6 +73,15 @@ func ResourceRecord() *schema.Resource {
 
 					return value
 				},
+				DiffSuppressFunc: func(k, oldValue, newValue string, d *schema.ResourceData) bool {
+					dnsZone := d.Get("dns_zone").(string)
+
+					// Normalize both values by removing trailing dots and domain suffix
+					normalizedOld := normalizeRecordName(oldValue, dnsZone)
+					normalizedNew := normalizeRecordName(newValue, dnsZone)
+
+					return normalizedOld == normalizedNew
+				},
 			},
 			"type": {
 				Type:             schema.TypeString,

internal/services/domain/record_test.go

@@ -910,3 +910,51 @@ func testAccCheckDomainRecordDestroy(tt *acctest.TestTools) resource.TestCheckFu
 		return nil
 	}
 }
+
+func TestAccDomainRecord_NameDiffSuppress(t *testing.T) {
+	tt := acctest.NewTestTools(t)
+	defer tt.Cleanup()
+
+	testDNSZone := "test-name-diff." + acctest.TestDomain
+	logging.L.Debugf("TestAccDomainRecord_NameDiffSuppress: test dns zone: %s", testDNSZone)
+
+	resource.ParallelTest(t, resource.TestCase{
+		PreCheck:                 func() { acctest.PreCheck(t) },
+		ProtoV6ProviderFactories: tt.ProviderFactories,
+		CheckDestroy:             testAccCheckDomainRecordDestroy(tt),
+		Steps: []resource.TestStep{
+			{
+				Config: fmt.Sprintf(`
+					resource "scaleway_domain_record" "dmarc" {
+						dns_zone = "%s"
+						name     = "_dmarc"
+						type     = "TXT"
+						data     = "v=DMARC1; p=none"
+					}
+				`, testDNSZone),
+				Check: resource.ComposeTestCheckFunc(
+					testAccCheckDomainRecordExists(tt, "scaleway_domain_record.dmarc"),
+					resource.TestCheckResourceAttr("scaleway_domain_record.dmarc", "name", "_dmarc"),
+					resource.TestCheckResourceAttr("scaleway_domain_record.dmarc", "type", "TXT"),
+					resource.TestCheckResourceAttr("scaleway_domain_record.dmarc", "data", "v=DMARC1; p=none"),
+				),
+			},
+			{
+				// Use FQDN format - should not cause replacement
+				Config: fmt.Sprintf(`
+					resource "scaleway_domain_record" "dmarc" {
+						dns_zone = "%s"
+						name     = "_dmarc.%s."
+						type     = "TXT"
+						data     = "v=DMARC1; p=none"
+					}
+				`, testDNSZone, testDNSZone),
+				Check: resource.ComposeTestCheckFunc(
+					testAccCheckDomainRecordExists(tt, "scaleway_domain_record.dmarc"),
+					resource.TestCheckResourceAttr("scaleway_domain_record.dmarc", "name", "_dmarc"),
+					resource.TestCheckResourceAttr("scaleway_domain_record.dmarc", "type", "TXT"),
+				),
+			},
+		},
+	})
+}