Added RRSet comments support

Author: Stanislau Petrusiou

powerdns/client.go

@@ -178,11 +178,17 @@ type Record struct {
 
 // ResourceRecordSet represents a PowerDNS RRSet object
 type ResourceRecordSet struct {
-	Name       string   `json:"name"`
-	Type       string   `json:"type"`
-	ChangeType string   `json:"changetype"`
-	TTL        int      `json:"ttl"` // For API v1
-	Records    []Record `json:"records,omitempty"`
+	Name       string    `json:"name"`
+	Type       string    `json:"type"`
+	ChangeType string    `json:"changetype"`
+	TTL        int       `json:"ttl"` // For API v1
+	Records    []Record  `json:"records,omitempty"`
+	Comments   []Comment `json:"comments,omitempty"`
+}
+
+type Comment struct {
+	Content string `json:"content,omitempty"`
+	Account string `json:"account,omitempty"`
 }
 
 type zonePatchRequest struct {

powerdns/resource_powerdns_record.go

@@ -57,6 +57,27 @@ func resourcePDNSRecord() *schema.Resource {
 				ForceNew:    true,
 				Description: "For A and AAAA records, if true, create corresponding PTR.",
 			},
+
+			"comment": {
+				Type: schema.TypeSet,
+				Elem: &schema.Resource{
+					Schema: map[string]*schema.Schema{
+						"content": {
+							Type:     schema.TypeString,
+							Required: true,
+							ForceNew: true,
+						},
+						"account": {
+							Type:     schema.TypeString,
+							Required: true,
+							ForceNew: true,
+						},
+					},
+				},
+				Optional:    true,
+				ForceNew:    true,
+				Description: "A comment about an RRSet.",
+			},
 		},
 	}
 }
@@ -70,6 +91,21 @@ func resourcePDNSRecordCreate(d *schema.ResourceData, meta interface{}) error {
 		TTL:  d.Get("ttl").(int),
 	}
 
+	comments := d.Get("comment").(*schema.Set).List()
+	if len(comments) > 0 {
+		commentObjs := make([]Comment, 0, len(comments))
+		for _, comment := range comments {
+			commentMap := comment.(map[string]interface{})
+			commentObjs = append(
+				commentObjs,
+				Comment{
+					Content: commentMap["content"].(string),
+					Account: commentMap["account"].(string),
+				})
+		}
+		rrSet.Comments = commentObjs
+	}
+
 	zone := d.Get("zone").(string)
 	ttl := d.Get("ttl").(int)
 	recs := d.Get("records").(*schema.Set).List()

powerdns/resource_powerdns_record_test.go

@@ -394,6 +394,22 @@ func TestAccPDNSRecord_TXT(t *testing.T) {
 	})
 }
 
+func TestAccPDNSRecord_WithComments(t *testing.T) {
+	resource.Test(t, resource.TestCase{
+		PreCheck:     func() { testAccPreCheck(t) },
+		Providers:    testAccProviders,
+		CheckDestroy: testAccCheckPDNSRecordDestroy,
+		Steps: []resource.TestStep{
+			{
+				Config: testPDSNRecordWithComments,
+				Check: resource.ComposeTestCheckFunc(
+					testAccCheckPDNSRecordExists("powerdns_record.test-comments"),
+				),
+			},
+		},
+	})
+}
+
 func TestAccPDNSRecord_ALIAS(t *testing.T) {
 	resourceName := "powerdns_record.test-alias"
 	resourceID := `{"zone":"sysa.xyz.","id":"alias.sysa.xyz.:::ALIAS"}`
@@ -654,3 +670,22 @@ resource "powerdns_record" "test-soa" {
 	ttl = 3600
 	records = [ "something.something. hostmaster.sysa.xyz. 2019090301 10800 3600 604800 3600" ]
 }`
+
+const testPDSNRecordWithComments = `
+resource "powerdns_record" "test-comments" {
+	zone = "sysa.xyz."
+	name = "comment.sysa.xyz."
+	type = "A"
+	ttl = 60
+	records = [ "1.1.1.1" ]
+
+    comment { 
+      content = "Test comment #1"
+      account = "Test account #1"
+    }
+
+    comment { 
+      content = "Test comment #2"
+      account = "Test account #2"
+    }
+}`

website/docs/r/record.html.markdown

@@ -42,6 +42,30 @@ resource "powerdns_record" "foobar" {
 }
 ```
 
+### Record with comments
+An example creating a record with comments:
+
+```hcl
+# Add a record with comments
+resource "powerdns_record" "foobar" {
+  zone    = "example.com."
+  name    = "www.example.com."
+	type = "A"
+	ttl = 60
+	records = [ "1.1.1.1" ]
+
+    comment { 
+      content = "Example comment #1"
+      account = "Example account #1"
+    }
+
+    comment { 
+      content = "Example comment #2"
+      account = "Example account #2"
+    }
+}
+```
+
 ### MX record example
 The following example shows, how to setup MX record with a priority of `10`.
 Please note that priority is not set as other `powerdns_record` properties; rather, it's part of the string that goes into `records` list.
@@ -145,6 +169,9 @@ The following arguments are supported:
 * `ttl` - (Required) The TTL of the record.
 * `records` - (Required) A string list of records.
 * `set_ptr` - (Optional) [**_Deprecated in PowerDNS 4.3.0_**] A boolean (true/false), determining whether API server should automatically create PTR record in the matching reverse zone. Existing PTR records are replaced. If no matching reverse zone, an error is thrown.
+* `comment` - (Optional) A comment about an RRSet.
+    *  `content` - (Required) The content of the comment.
+    *  `account` - (Required) The account of the comment.
 
 ### Attribute Reference